Projet

Général

Profil

Bug #18199

Ne pas logguer certains problèmes de géoloc

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
29 août 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Lié à w.c.s. - Development #18172: wcs vs configuration de loggingFermé25 août 2017

Actions

Révisions associées

Révision 459fa64d (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

workflow: do not log as error some geolocation failures (#18199)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

#2

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).

#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

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.

#4

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Ack.

#5

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)
#6

Mis à jour par Frédéric Péters il y a plus de 6 ans

#7

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

Formats disponibles : Atom PDF