https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342020-01-29T14:06:52ZRedmine Entr’ouvertdjango-mellon - Support #39382: Ne pas logger un fichier de métadonné invalide comme une erreurhttps://dev.entrouvert.org/issues/39382?journal_id=2126172020-01-29T14:06:52ZBenjamin Dauvergne
<ul><li><strong>Description</strong> mis à jour (<a title="Voir les différences" href="/journals/212617/diff?detail_id=186874">diff</a>)</li></ul> django-mellon - Support #39382: Ne pas logger un fichier de métadonné invalide comme une erreurhttps://dev.entrouvert.org/issues/39382?journal_id=2126382020-01-29T14:28:35ZThomas Noël
<ul><li><strong>Assigné à</strong> mis à <i>Thomas Noël</i></li></ul> django-mellon - Support #39382: Ne pas logger un fichier de métadonné invalide comme une erreurhttps://dev.entrouvert.org/issues/39382?journal_id=2126432020-01-29T14:37:03ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Information nécessaire</i></li><li><strong>Assigné à</strong> changé de <i>Thomas Noël</i> à <i>Benjamin Dauvergne</i></li></ul><p>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.</p>
<pre>
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
</pre>
<p>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 ?</p> django-mellon - Support #39382: Ne pas logger un fichier de métadonné invalide comme une erreurhttps://dev.entrouvert.org/issues/39382?journal_id=2126682020-01-29T15:14:31ZBenjamin Dauvergne
<ul></ul><p>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.</p> django-mellon - Support #39382: Ne pas logger un fichier de métadonné invalide comme une erreurhttps://dev.entrouvert.org/issues/39382?journal_id=2225862020-03-24T20:36:05ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Information nécessaire</i> à <i>Rejeté</i></li></ul>Je ne comprends pas ce qui est voulu ici :
<ul>
<li>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)</li>
<li>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)</li>
</ul>