Projet

Général

Profil

Development #13881

ne pas utiliser les settings pour garder les métadonnées en cache

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

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

Lié à django-mellon - Development #26143: ne pas faire de cache des métadonnées localesFermé05 septembre 2018

Actions

Révisions associées

Révision a1fa7d36 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

don't cache local metadata anymore (#13881)

Historique

#1

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.

#3

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.

#4

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

#5

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

Pour laisser #26143 faire sa vie, j'attache ici ce patch.

#6

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

#7

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

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF