Projet

Général

Profil

Development #29235

remplacer statics_hash par un simple start_timestamp

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

Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
20 décembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

On calcule un hash des versions des différents modules Publik installé et on utilise ça en query string de fichiers statiques, pour éviter le cache côté navigateur.

Côté gadjo on a arrêté cette idée, on met juste la timestamp du démarrage du service.

On ferait ça aussi pour Combo ? (c'est dans gadjo donc le code est déjà disponible partout)


Demandes liées

Lié à Intégrations graphiques Publik - Development #29238: utiliser ?{%start_timestamp%} de gadjo pour les fichiers statiquesFermé20 décembre 2018

Actions

Historique

#1

Mis à jour par Thomas Noël il y a plus de 5 ans

Tu vois un possible inconvénient ? (moi pas, et le code de start_timestamp est tout simple)

À terme ça pourrait nous faire gagner un context_processor de moins (hobo.context_processors.statics_hash) dans combo, donc rien que pour ça je dirais oui. À cette occasion, penser à ajouter une dépendance sur gadjo dans publik-base-theme (parce qu'a priori c'est dans publik-base-theme qu'est utilisé statics_hash, pas vraiment dans combo, non ?).

#2

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

Tu vois un possible inconvénient ?

Les restart de combo invalideront nécessairement le cache alors qu'avant, s'il n'y avait pas eu de changement de versions dans les paquets, ce n'était pas le cas. Dans la pratique, non, pas d'inconvénient.

pas vraiment dans combo

C'est dans combo qu'on ajoute le context processor,

combo/wsgi.py:if 'hobo.context_processors.statics_hash' in settings.TEMPLATES[0]['OPTIONS']['context_processors']:
combo/wsgi.py:    # requests via the statics_hash template var and we prefer not to
debian/debian_config.py:# add statics_hash context processor
debian/debian_config.py:TEMPLATES[0]['OPTIONS']['context_processors'].append('hobo.context_processors.statics_hash')

Mais après, oui, il s'agirait également de modifier publik-base-theme (j'ajouterai la dépendance mais c'est factice, ça reste aux applications à avoir gadjo dans leur INSTALLED_APPS).

Et après encore, on pourra virer le context processor d'Hobo.

#3

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

  • Lié à Development #29238: utiliser ?{%start_timestamp%} de gadjo pour les fichiers statiques ajouté
#4

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

Mais après, oui, il s'agirait également de modifier publik-base-theme

Et plutôt avant qu'après, voilà dans #29238. Ensuite modifier combo. Ensuite modifier hobo.

Formats disponibles : Atom PDF