Projet

Général

Profil

Anomalie #3979

Les CSS dans le cache sont en double.

Ajouté par RealET 🔸 il y a plus d'un an. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Haut
Assigné à:
-
Version cible:
Début:
06/08/2017
Echéance:
% réalisé:

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 ko) RealET 🔸, 06/08/2017 15:23

347c3aa8c26c253b0d302db64a7d9e8e.css Voir (36,4 ko) RealET 🔸, 06/08/2017 15:23

e04986d28f7e95243b1968b303c32770.css Voir (36,4 ko) RealET 🔸, 06/08/2017 15:23

e04986d28f7e95243b1968b303c32770.css.gz (8,11 ko) RealET 🔸, 06/08/2017 15:23

3979.patch Voir (2,05 ko) b b, 09/01/2018 21:08

Historique

#1 Mis à jour par b b il y a plus d'un an

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 Mis à jour par RealET 🔸 il y a plus d'un an

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

#3 Mis à jour par jluc - il y a plus d'un an

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 Mis à jour par Fil _ il y a plus d'un an

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 Mis à jour par b b il y a environ un an

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

#6 Mis à jour par b b il y a environ un an

  • Statut changé de Nouveau à En cours

#7 Mis à jour par cedric - il y a 6 mois

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

#8 Mis à jour par cedric - il y a 6 mois

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

#9 Mis à jour par cedric - il y a 6 mois

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 Mis à jour par cedric - il y a 6 mois

  • Statut changé de En cours à Fermé
  • Resolution mis à invalid

documenté par r111775

Formats disponibles : Atom PDF