Bug #18199
Ne pas logguer certains problèmes de géoloc
0%
Description
C'est un schéma tout à fait autorisé d'avoir une série d'appel à l'action de géoloc, et certains échecs sont "normaux", provoqués par les données de l'usager et non par une défaillance du système (genre une image sans donnée exif).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-workflow-do-not-log-as-error-some-geolocation-failur.patch 0001-workflow-do-not-log-as-error-some-geolocation-failur.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Mouais je ne sais pas, j'irai jusqu'au niveau debug pour les 3, c'est le genre d'informations qui pourrissent les logs sans apporter vraiment de détails pertinents (et je dois faire le ménage de mon coté dans authentic aussi), idéalement il faudrait deux loggers sinon un technique et un fonctionnel (le premier log ces détails au niveaux qu'on souhaite error si il faut, le deuxième ne log que les actions haut niveau des utilisateurs et agents). Disons un logger nommé wcs.users et un wcs.internals, on sort le nom du logger dans les lignes de log et dans la vue des logs on n'afficherait par défaut que wcs.users (et je me dis aussi qu'à un moment on devrait prévoir un logger SQL).
Donc là tout de suite je dirai get_logger().debug() sauf si on m'indique que c'est vraiment important de savoir ce qu'il se passe au niveau de la géoloc en toute circonstance (j'ai conscience du niveau d'hystérie de certains clients sur ce sujet mais on doit l'ignorer je pense).
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-workflow-do-not-log-as-error-some-geolocation-failur.patch 0001-workflow-do-not-log-as-error-some-geolocation-failur.patch ajouté
Voilà en niveau debug.
Sur des évolutions de fond sur le logging, notamment la séparation technique/fonctionnel, il y a aussi des commentaires dans le très vieux #1862.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 459fa64dafa5e55713b694340410bd47c50c56f2 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Aug 29 08:00:22 2017 +0200 workflow: do not log as error some geolocation failures (#18199)
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Lié à Development #18172: wcs vs configuration de logging ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflow: do not log as error some geolocation failures (#18199)