Project

General

Profile

Développement #50975

Améliorer la journalisation de la connexion après création de compte

Added by Valentin Deniaud almost 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
09 February 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Quand on se crée un compte, on est automatiquement connecté et dans le journal ça fait :

9 février 2021 11:29     Jean Dupont     vtv6zf     authentification via email
9 février 2021 11:29     Jean Dupont     -     création de compte via email

Ce n'est pas une info très utile je pense, et ça perturbe la lecture des statistiques de connexion.

Pour améliorer ça, on pourrait ne pas journaliser ce qui passe par simulate_authentication() ? Ou plutôt aller cacher les connexions dont la méthode est 'email' au niveau des stats ? Ou plutôt changer ce 'email' qui semble être une valeur par défaut, que la valeur par défaut soit plutôt 'password' ?


Files

Associated revisions

Revision d4124247 (diff)
Added by Valentin Deniaud almost 4 years ago

utils: do not record login if simulated (#50975)

History

#2

Updated by Benjamin Dauvergne almost 4 years ago

Je penche plutôt pour « Pour améliorer ça, on pourrait ne pas journaliser ce qui passe par simulate_authentication() ? » avec un paramètre logging=True par défaut, désactivable dans ce cas là.

#3

Updated by Valentin Deniaud almost 4 years ago

  • Assignee set to Valentin Deniaud
#4

Updated by Valentin Deniaud almost 4 years ago

#5

Updated by Benjamin Dauvergne almost 4 years ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Valentin Deniaud almost 4 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit f956b5f04810a2d94767edf32f1c3ab180bf1437
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Feb 9 17:13:41 2021 +0100

    utils: do not record login if simulated (#50975)
#7

Updated by Frédéric Péters almost 4 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF