Evolution #1031
closed
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.
Files
Updated by Johan Cwiklinski over 5 years ago
- Due date set to 05/26/2017
- Start date changed from 05/27/2017 to 05/26/2017
- Follows Evolution #1029: Adresses IP navigateur dans logs d'échecs d'authentification added
Updated by Georges Racinet over 5 years ago
- File 0001-Optionally-use-X-Forwarded-For-in-galette_logs-fixes.patch 0001-Optionally-use-X-Forwarded-For-in-galette_logs-fixes.patch added
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.
Updated by Johan Cwiklinski over 5 years ago
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 ;)
Updated by Georges Racinet over 5 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
Appliqué par commit b05eeb9bf1ff7b96419d908435f91965355431d7.
Updated by Johan Cwiklinski over 5 years ago
- Status changed from Résolu to 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.