Je pense avoir vu quelque part qu'un certain browser (je croix etre IE, mais pas sur de la version) n'accepte pas le header Location avec une url relative. Il faut qu'elle soit absolue (conformement aux specs http). Ca peut devenir assez problematique dans ce cas... (generation des urls complets - c'est toujours la galere pour gerer tous les types/configs des serveurs ..._

rvelices a écrit:

Je pense avoir vu quelque part qu'un certain browser (je croix etre IE, mais pas sur de la version) n'accepte pas le header Location avec une url relative. Il faut qu'elle soit absolue (conformement aux specs http). Ca peut devenir assez problematique dans ce cas... (generation des urls complets - c'est toujours la galere pour gerer tous les types/configs des serveurs ..._

Effectivement mais ce n'est pas difficile de faire une redirection avec une url absolue ce que je fais déjà sur mon site.
J'ai donné mon avis sur la façon de corriger le problème et je vois que vous campez sur vos positions. Corrigez-le comme bon vous semble. Je ferais un patch pour mon site pour ne pas avoir cette page inutile.

z0rglub a écrit:

Comme quoi, tout est question de point de vue :-)

On fait un petit vote de l'équipe pour savoir si ca passe en MOD ou dans la version Alligator?

rvelices a écrit:

On peut faire 2 fonctions: redirect_http et redirect_html.
On laisse l'ancienne fonction redirect utiliser soit l'une soit l'autre (redirect_http dans le cas du pb actuel quand le template n'est pas encore defini, redirect_html pour les fonctions page_not_found et access_denied et au choix dans le fichier de config dans les autres cas).
Apres dans le temps on voit si ca pose probleme ou non et ca sera facile a changer. Les developpeurs pourront utiliser une config redirect_html pour analyser les requetes sql par exemple.

Vendu pour moi!

rvelices a écrit:

On peut faire 2 fonctions: redirect_http et redirect_html

Excellent! Le diaporama utilise directement redirect_html, ailleurs c'est redirect qui appelle soit redirect_html soit redirect_http selon un paramètre de configuration. Question subsidiaire : par défaut, on utilise le redirect_html ou redirect_http ? Mon avis serait plutôt qu'on utilise redirect_html pour éviter les soucis, mais on pourra dire aux utilisateurs avancés comment passer en mode redirect_http. Si on fait le contraire, cela voudra dire que des utilisateurs plutôt débutant (qui font des erreurs dans les fichiers inclus) doivent éditer la conf.

Sur le coup, j'étais partir sur du 1.b mais après réflexion pourquoi pas du 1.a puisque par défaut rien ne change!

Dernière modification par rub (2006-10-17 23:57:34)

Nicolas, est-ce que tu t'en charge de faire les 2 fonctions et eventuellement faire un rollback sur la 1.6 ?

Sinon je peux le faire moi en Alligator, mais je me sens pas a l'aise pour un revert svn surtout qu'il y a eu d'autres commit depuis sur la 1.6. Pierrick, est-ce que t'as une solution magique? Au pire de cas je peux reporter mes modifs de l'Alligator vers 1.6.

z0rglub a écrit:

Cela dit, je ne vois pas du tout l'intérêt de backporter une fonctionnalité. La politique de gestion des branches, c'est de ne backporter que les bugs bloquants ou triviaux ou sans contournement. Les fonctionnalités (même technique comme ici) n'entrent pas dans ces critères.

D'accord sur ce point, mais dans l'etat actuel (branche 1.6 contient deja le auto_login et ce commit a eu pas mal des modifs) ca me parrait plus simple et moins risque de continuer a corriger la 1.6 que de faire un roll back.