Development #13881
ne pas utiliser les settings pour garder les métadonnées en cache
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
07 novembre 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
cache = getattr(settings, '_MELLON_METADATA_CACHE', {}) if not entity_id in cache: login_url = reverse(app_settings.LOGIN_URL)
Ça oblige à redémarrer Combo si jamais les métadonnées changent; ce serait bien d'utiliser django.core.cache, pour permettre une expiration, pour permettre aussi de vider le cache par un simple redémarrage de memcached.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 5 ans
Pire : si le combo est "par hasard" accessible et interrogé en http, les données en cache se retrouve en http... sur le worker en question.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Nouveau à Rejeté
Rendu obsolète par #26143.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Development #26143: ne pas faire de cache des métadonnées locales ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-don-t-cache-local-metadata-anymore-13881.patch 0001-don-t-cache-local-metadata-anymore-13881.patch ajouté
- Statut changé de Rejeté à Solution proposée
- Patch proposed changé de Non à Oui
Pour laisser #26143 faire sa vie, j'attache ici ce patch.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Dans #26143, le patch a été validé.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a1fa7d36dfdec71de154f4713471c901549a5e61 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Sep 5 16:49:23 2018 +0200 don't cache local metadata anymore (#13881)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
don't cache local metadata anymore (#13881)