Projet

Général

Profil

Anomalie #1991

redirection en boucle sur eole-web

Ajouté par Emmanuel GARETTE (2) il y a plus de 12 ans. Mis à jour il y a plus de 12 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Début:
25/08/2011
Echéance:
% réalisé:

100%

Temps passé:
Distribution:
EOLE 2.3

Description

La valeur de web_redirection est à "/" par défaut :

            <variable name='web_redirection' type='string' description='Application web par défaut (redirection)' >
                <value>/</value>
            </variable>

Dans le template nous avons :

%if %%is_defined('activer_envole') and %%activer_envole == 'oui'
 RewriteRule ^/$ %%alias_envole [R=permanent]
%else
 RewriteRule ^/$ %%web_redirection [R=permanent]
%end if

Ce qui signifie que la ligne sera :
RewriteRule ^/$ / [R=permanent]

(si on demande / on renvoi vers ... /).

Il faudrait faire un test pour éviter la redirection en boucle :

-- Vhost.conf.old  2011-08-25 08:49:32.533349685 +0200
+++ Vhost.conf  2011-08-25 08:54:58.433022085 +0200
@@ -7,8 +7,12 @@
  ErrorLog /var/log/apache2/error.log
  RewriteEngine On
 %if %%is_defined('activer_envole') and %%activer_envole == 'oui'
+ %if %%alias_envole != '/'
  RewriteRule ^/$ %%alias_envole [R=permanent]
+ %end if
 %else
+ %if %%web_redirection != '/'
  RewriteRule ^/$ %%web_redirection [R=permanent]
+ %end if
 %end if
 </VirtualHost>
--- Vhost-ssl.conf.old  2011-08-25 08:50:23.000750923 +0200
+++ Vhost-ssl.conf  2011-08-25 08:53:46.867051833 +0200
@@ -12,9 +12,13 @@
  ServerName %%nom_machine
  RewriteEngine On
 %if %%is_defined('activer_envole') and %%activer_envole == 'oui'
+ %if %%alias_envole != '/'
  RewriteRule ^/$ %%alias_envole [R=permanent]
+ %end if
 %else
+ %if %%web_redirection != '/'
  RewriteRule ^/$ %%web_redirection [R=permanent]
+ %end if
 %end if
 #  <Location />
 #    Order Allow,Deny

De plus, je ne comprends pas bien pourquoi le template est si compliqué. alias_envole et web_redirection sont les mêmes variables. Pourquoi ne pas jouer avec les redefines ?

Révisions associées

Révision 55b10b74 (diff)
Ajouté par Walid Nouh il y a plus de 14 ans

see #1991 Add new ldap fields

Révision 81c7abc1 (diff)
Ajouté par Walid Nouh il y a plus de 14 ans

see #1991 : add import_externalauth_users right

Révision 3546b388 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 First work on ldap user import from ticket form

Révision 06036f3a (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix ldap search issues

Révision 92e529ba (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix ldap import

Révision 60ab0319 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Add missing file and new checks for popup

Révision 47738572 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Display criterias 2 per line (like autoreport)

Révision 3c46d802 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix basedn retrieval

Révision 75bb4b94 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improvements + correct ldapImportUSerByServerId call

Révision 6c815764 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Add interface expert mode

Révision bc7763fd (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improve ldap link

Révision 357225b7 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improve last commit

Révision 3a4355e5 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Look for user with phone and email in user dropdown

Révision 181097af (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improve search GUI

Révision 5a00db93 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improve GUI

Révision 6fc5c7b8 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Use session to store data instead of $_REQUEST

Révision ec745515 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Improve previous commit

Révision 42cfa259 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix basedn search criteria

Révision 866cd031 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Remove addldapuser.php page

Révision 7f691a07 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Add basedn option when searching in expert mode + clean code

Révision 951d3ef4 (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix bugs + add is_default to glpi_authldaps

Révision e74e3feb (diff)
Ajouté par Walid Nouh il y a environ 14 ans

see #1991 Fix right issues

Révision 78d90dd8 (diff)
Ajouté par Joël Cuissinat il y a plus de 12 ans

prise en compte du risque de redirection web en boucle (fixes #1991)

Historique

#1 Mis à jour par Joël Cuissinat il y a plus de 12 ans

  • Version cible mis à Mises à jour 2.3 - 02 RC

#2 Mis à jour par Joël Cuissinat il y a plus de 12 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#3 Mis à jour par Joël Cuissinat il y a plus de 12 ans

  • Statut changé de Résolu à Fermé
  • Distribution mis à EOLE 2.3
root@horusng:~# echo $web_redirection
/
root@horusng:~# grep RewriteRule /usr/share/eole/creole/distrib/Vhost*.conf
/usr/share/eole/creole/distrib/Vhost.conf: RewriteRule ^/$ %%alias_envole [R=permanent]
/usr/share/eole/creole/distrib/Vhost.conf: RewriteRule ^/$ %%web_redirection [R=permanent]
/usr/share/eole/creole/distrib/Vhost-ssl.conf: RewriteRule ^/$ %%alias_envole [R=permanent]
/usr/share/eole/creole/distrib/Vhost-ssl.conf: RewriteRule ^/$ %%web_redirection [R=permanent]
root@horusng:~# grep RewriteRule /etc/apache2/sites-enabled/Vhost*.conf
root@horusng:~#

Formats disponibles : Atom PDF