Project

General

Profile

Development #29235

remplacer statics_hash par un simple start_timestamp

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Start date:
20 Dec 2018
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

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)


Related issues

Related to Intégrations graphiques Publik - Development #29238: utiliser ?{%start_timestamp%} de gadjo pour les fichiers statiques Solution déployée 20 Dec 2018

History

#1 Updated by Thomas Noël 4 months ago

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 Updated by Frédéric Péters 4 months ago

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 Updated by Frédéric Péters 4 months ago

  • Related to Development #29238: utiliser ?{%start_timestamp%} de gadjo pour les fichiers statiques added

#4 Updated by Frédéric Péters 4 months ago

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.

Also available in: Atom PDF