Evolution #1031
ferméPrise en compte des proxy dans les logs
100%
Description
La table des logs stocke un certain nombre d'évènements ; notamment les tentatives infructueuses de connexion.
Actuellement, l'adresse IP est enregistrée, mais si l'instance de Galette se trouve derrière un proxy, c'est l'adresse du proxy qui sera enregistrée, et non celle de l'utilisateur.
Fichiers
Mis à jour par Johan Cwiklinski il y a plus de 7 ans
- Echéance mis à 26/05/2017
- Début changé de 27/05/2017 à 26/05/2017
- Suit Evolution #1029: Adresses IP navigateur dans logs d'échecs d'authentification ajouté
Mis à jour par Georges Racinet il y a plus de 7 ans
- Fichier 0001-Optionally-use-X-Forwarded-For-in-galette_logs-fixes.patch 0001-Optionally-use-X-Forwarded-For-in-galette_logs-fixes.patch ajouté
Voici un patch qui fournit la fonctionnalité, en utilisant une nouvelle option de config.inc.php
Je ne savais pas trop quelles règles de nommage appliquer pour ça, ni où documenter proprement la nouvelle option.
C'est donc une version à affiner, à d'autant mieux relire que je ne suis pas très à l'aise en PHP.
Mis à jour par Johan Cwiklinski il y a plus de 7 ans
Hello,
J'ai jetté un rapide coup d'oeil ; ça me semble pas mal du tout :-)
Je pense que ce devrait être documenté dans cette partie de la doc : http://galette.readthedocs.io/fr/develop/usermanual/avancee.html (https://git.tuxfamily.org/galette/galettedoc.git/tree/source/usermanual/avancee.rst?h=develop).
Je vais essayer d'y regarder de plus près rapidement, la nouvelle fonction devrait être utilisée pour le #1029 également. Merci ;)
Mis à jour par Georges Racinet il y a plus de 7 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit b05eeb9bf1ff7b96419d908435f91965355431d7.
Mis à jour par Johan Cwiklinski il y a plus de 7 ans
- Statut changé de Résolu à Fermé
J'ai intégré le patch, merci.
Juste quelques modifications :
- déplacement dans le fichier behavior (le fichier de config est généré, l'utilisateur ne doit pas y toucher)
- renommage avec un préfixe GALETTE_ (comme ça, pas de soucis ;)).
J'ai également ajouté ça à la documentation.