Project

General

Profile

Anomalie #3979

Les CSS dans le cache sont en double.

Added by RealET 🔸 about 2 years ago. Updated about 1 year ago.

Status:
Fermé
Priority:
Haut
Assignee:
-
Target version:
Start date:
08/06/2017
Due date:
% Done:

0%

Resolution:
invalid
Navigateur:

Description

Bonjour,

En investiguant sur un problème de cache, j'ai fait une installation vierge de SPIP 3.1 SVN (23679)
Pour constater que si j'active la compression des CSS et des JS, dans le dossier local/cache-css il y a 2 fois le fichier compressé .css et aussi 2 fois le.css.gz
Et ils sont identiques ! (ci-joints).

347c3aa8c26c253b0d302db64a7d9e8e.css.gz (8.11 KB) RealET 🔸, 08/06/2017 03:23 PM

347c3aa8c26c253b0d302db64a7d9e8e.css View (36.4 KB) RealET 🔸, 08/06/2017 03:23 PM

e04986d28f7e95243b1968b303c32770.css View (36.4 KB) RealET 🔸, 08/06/2017 03:23 PM

e04986d28f7e95243b1968b303c32770.css.gz (8.11 KB) RealET 🔸, 08/06/2017 03:23 PM

3979.patch View (2.05 KB) b b, 01/09/2018 09:08 PM

History

#1 Updated by b b about 2 years ago

Il faudrait nous fournir un peu plus d'infos sur le contexte du problème, par exemple, quel est le squelette qui génère/appelle tes CSS ?

#2 Updated by RealET 🔸 about 2 years ago

Je pensais l'avoir dit : c'est la dist de SPIP (« SPIP vierge »).
Et c'est la page d'accueil du site.

#3 Updated by jluc - about 2 years ago

Sur un site en prod, je vois qu'il y a quelques fichiers de cache-css en double et apparemment identiques.
Par exemple le cache pour prive/themes/spip/reset.css ou squelette-dist/css/clear.css (qui n'apparaissent pas concaténés par ailleurs ; normal ?).
Pour certains fichiers il y a plusieurs versions mais c'est parceque j'ai édité les sources css entre temps.

#4 Updated by Fil _ about 2 years ago

Deux questions imbriquées :

1) on a le .css et le .css.gz car ça pouvait avoir un impact (hypothétique?) sur la vitesse de compression d'apache. A mon avis ça fait plus de mal que de bien et on devrait supprimer la version .gz.

2) deux copies sous des numéros différents : là, je ne sais pas. Vérifier s'ils sont créés par le même process et s'ils sont vraiment identiques ?

#5 Updated by b b almost 2 years ago

Voilà déjà un patch pour le premier point, retrait de la génération des versions .gz et de leur mention.

#6 Updated by b b almost 2 years ago

  • Status changed from Nouveau to En cours

#7 Updated by cedric - about 1 year ago

Tu veux qu'on te donnes les droits de commits b_b ?:D

#8 Updated by cedric - about 1 year ago

le patch pour supprimer les gz est déà intégré par r108432

#9 Updated by cedric - about 1 year ago

J'adore la priorité "Haut" du genre "ça va tout casser si on laisse comme ça" :)
Donc c'est pas un bug mais une feature tout Ă  fait normale
cf https://core.spip.net/projects/compresseur/repository/entry/inc/compresseur_concatener.php#L131

Il y a une $final_callback chargee de regrouper les @import au début de la CSS concaténée, et qui, éventuellement, ne change rien quand tu n'as pas d'@import donc on créé un fichier intermédiaire temporaire sur lequel appliquer la callback finale.

Et ce fichier intermédiaire n'est pas supprimé car si on a 2 process concurrents qui font le même job (car lancés a peu de temps de décalage) on a le risque que le second essaye d'appliquer la callback finale alors que le premier vient de supprimer le fichier temporaire

#10 Updated by cedric - about 1 year ago

  • Status changed from En cours to FermĂ©
  • Resolution set to invalid

documenté par r111775

Also available in: Atom PDF