Project

General

Profile

Support #39382

Ne pas logger un fichier de métadonné invalide comme une erreur

Added by Benjamin Dauvergne 2 months ago. Updated 6 days ago.

Status:
Rejeté
Priority:
Normal
Target version:
-
Start date:
29 Jan 2020
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

History

#1 Updated by Benjamin Dauvergne 2 months ago

  • Description updated (diff)

#2 Updated by Thomas Noël 2 months ago

  • Assignee set to Thomas Noël

#3 Updated by Thomas Noël 2 months ago

  • Assignee changed from Thomas Noël to Benjamin Dauvergne
  • Status changed from Nouveau to Information nécessaire

En fait on loggue en erreur l'absence de métadonnées (quand elles sont téléchargées), mais on warning quand un fichier est absent... C'est un peu le bronx.

  111         warning = logger.warning
  112         if 'METADATA' not in idp:
  113             # if we have no metadata in cache, we must emit errors
  114             warning = logger.error

Est-ce qu'on passe en warning partout tout le temps ? Des soucis sur les métadonnées ça me semble quand même un problème (surtout quand elles sont sur un fichier local) et je trouve ça normal de recevoir une alerte de "haut niveau" (error). Non ?

#4 Updated by Benjamin Dauvergne 2 months ago

Ouais j'avais oublié ça ; c'est justement prévu pour ne pas nous pourrir d'erreur tant qu'on a un truc valable en cache, je ne sais plus si le cache expire ou pas. Le but c'est de cacher l'erreur si ça dure 24h pas 1 semaine.

#5 Updated by Benjamin Dauvergne 6 days ago

  • Status changed from Information nécessaire to Rejeté
Je ne comprends pas ce qui est voulu ici :
  • on log en erreur si les métadonnées doivent être téléchargées mais ne sont pas dispo en cache ou que le cache est trop vieux (24h)
  • on log en erreur si un fichier local de métadonnée n'est pas ouvrable (open foire) ou qu'il est mal formé (on n'y trouve pas les noeuds de base et l'entityID)

Also available in: Atom PDF