Anomalie #3204
wpad alors que eole-reverseproxy sans eole-proxy
Description
Si on installe eole-reverseproxy sans eole-proxy, les fichiers wpad sont générés.
J'ai fait un patch du dictionnaire sans savoir si la variable activer_proxy existe vraiment et si le dictionnaire est bien inférieur à 25_nginx.
Dans le template nginx.default on pourrait remplacer le test :
%if %%is_defined('container_ip_proxy')
Par activer_proxy (ou la variable équivalente).
Related issues
Associated revisions
- dicos/25_nginx.xml : ajout d'une filelist sur les fichiers wpad
Fixes #3204 @45m
Suppression des modifications ref #3204
History
#1 Updated by Joël Cuissinat almost 11 years ago
- Status changed from Nouveau to Accepté
- Target version set to Mises à jour 2.3.5 RC
- Estimated time set to 0.50 h
#2 Updated by Joël Cuissinat almost 11 years ago
- Project changed from conf-amon to eole-reverseproxy
#3 Updated by Joël Cuissinat almost 11 years ago
- Assigned To set to Joël Cuissinat
Le dico 23_proxy est bien chargé avant celui d'nginx mais, à ce jour : squid n'est pas désactivable !
J'ai donc fait le hidden en testant sur la présence de l'une des variable squid : activer_squid_auth en l'occurrence.
J'aurais préféré ne pas initialiser cette variable (pas de value) et faire le test comme ça mais je suis tombé sur #3351 et au final, je ne suis pas très satisfait de cette correction.
#4 Updated by Joël Cuissinat almost 11 years ago
- Status changed from Accepté to Résolu
- % Done changed from 0 to 100
Appliqué par commit 3088e5a0f4f732b4eb75a23d12aa623d84f4ba6e.
#5 Updated by Joël Cuissinat almost 11 years ago
- Status changed from Résolu to En attente d'informations
- % Done changed from 100 to 80
En attente de validation, mes modifications pourraient être supprimées sans problème !
#6 Updated by Joël Cuissinat almost 11 years ago
Emmanuel a supprimé mes modifications mais suite à la correction de #3351, il est possible de réintroduire ce genre de correction mais sans affecter de valeur à la variable redéfinie (activer_squid_auth dans notre cas).
#7 Updated by Joël Cuissinat almost 11 years ago
- Status changed from En attente d'informations to Résolu
- % Done changed from 80 to 100
Appliqué par commit 755e76c784b5172d80506d21a88b619d084cac00.
#8 Updated by Fabrice Barconnière almost 11 years ago
- Status changed from Résolu to Fermé