Anomalie #3557
Header composed-by tronqué
100%
Description
La chaîne composed-by insérée dans les header HTPP par SPIP est limitée à 934 caractères.
Résultat : sur des sites avec vraiment beaucoup de plugins, la liste des plugins est tronquée.
Du coup, les stats d'usage d'un plugin pourront être faussées sur http://stats.spip.net/ et http://plugins.spip.net/
Un exemple :
SPIP 3.1.0-beta @ www.spip.net + spip(3.1.0-beta),compagnon(1.5.1),dump(1.7.4),images(1.1.10),forum(1.9.29),jqueryui(1.11.4),mediabox(0.10.2),mots(2.7.3),organiseur(0.10.4),petitions(1.5.2),ancresdouces(1.4.6),boutonstexte(2.0.3),calendriermini(2.3.8),citations_bb(0.1.2),corbeille(3.0.6),crayons(1.23.0),facteur(3.1.2),fonctions_images(0.6.11),mailcrypt(2.4.2),memoization(1.5.7),nospam(1.5.13),notifications(3.5.2),orthotypo(1.4.1),palette(3.0.6),spip_bonux(3.2.1),typo_guillemets(1.5.2),yaml(1.5.2),porte_plume(1.15.7),revisions(1.8.4),safehtml(1.4.1),sites(1.9.12),squelettes_par_rubrique(1.1.1),stats(0.6.7),svp(0.86.10),tw(1.3.4),urls(1.5.2),vertebres(1.2.4),iterateurs(0.6.1),queue(0.6.6),breves(1.3.10),compresseur(1.9.9),medias(2.10.24),accessibilite(0.8.8),pp_chgt_lg(0.3.1),fulltext(1.1.5),oembed(1.4.10),typoenluminee(3.5.4),verifier(1.0.8),agenda(3.14.10),saisies(2.5.10),savecfg(3.0.0),uploadhtml5(1.3.1),dictionnaires(
Une idée comme ça : un var_mode=sad qui listerait les plugins installés (en séparant les plugins et plugins-dist)
Révisions associées
Fix #3557 : simplifier l'entête compsed-by en y indiquant l'url du fichier local/config.txt
Historique
#1
Mis à jour par RealET 🔸 il y a plus de 3 ans
Mis à jour par Je découvre (au moins en 3.1) : local/config.txt
Un peu plus difficile à trouver en mutualisation facile : sites/nomdedomaine/local/config.txt
Peut-être que pour améliorer les perfs en ayant un entête plus concis, le Composed-by pourrait ne contenir que :
SPIP 3.1.0-beta @ www.spip.net + URL du fichier config.txt
#2
Mis à jour par b b il y a plus de 3 ans
Mis à jour par - Sujet changé de http://stats.spip.net/ faussées à Header composed-by tronqué
#3
Mis à jour par RealET 🔸 il y a presque 3 ans
Autre outil impacté par cette limitation : http://blog.nursit.net/Administrer-les-plugins-d-un-site.html
#4
Mis à jour par RealET 🔸 il y a presque 3 ans
http://zone.spip.org/trac/spip-zone/changeset/97960 permet de chercher à prendre en compte les sites en mutualisation facile.
#5
Mis à jour par RealET 🔸 il y a environ 2 ans
En terme de #webperf mettre :
SPIP 3.1.0-beta @ www.spip.net + URL du fichier config.txt
diminuerait aussi la taille de ce qui circule...
#6
Mis à jour par b b il y a environ 2 ans
- Version cible mis à 3.1
#7
Mis à jour par b b il y a environ un an
- Statut changé de Nouveau à En cours
Pour info, stats.spip.net n'est pas impacté par ce comportement car il est bien conçu ;)
https://zone.spip.org/trac/spip-zone/browser/_dev_/univers_spip/inc/univers_analyser.php#L131
plugins.spip utilise les infos générées stats.spip.net, il n'est donc pas impacté non plus, amha on peut fermer le ticket.
#8
Mis à jour par RealET 🔸 il y a environ un an
C''est surtout que j'ai du rajouter https://zone.spip.org/trac/spip-zone/browser/_dev_/univers_spip/inc/univers_analyser.php#L135 à 139 pour que ça marche aussi dans le cas de mutu "assez" classiques.
Mais que ça reste du spécifique bricolé.
Je relance donc l'idée :
qui permettrait :Peut-être que pour améliorer les perfs en ayant un entête plus concis, le Composed-by pourrait ne contenir que :
SPIP 3.1.0-beta @ www.spip.net + URL du fichier config.txt
- d'être générique
- de gagner pas mal d'octets dans ce qui est envoyé en entête HTTP
#9
Mis à jour par b b il y a environ un an
L'idée d'envoyer l'url du fichier config.txt est intéressante, je vote pour son intégration, mais elle ne résoudra en rien le problème spécifique aux sites en mutu puisqu'on annoncera quelque chose comme https://www.spip.net/local/config.txt
Ça restera donc du code bricolé pour les mutus...
#10
Mis à jour par b b il y a environ un an
- Fichier 3557.patch Voir ajouté
Voici un patch qui intègre la proposition et qui avec de la chance affichera tout de même la bonne url sur une mutu.
#11
Mis à jour par RealET 🔸 il y a environ un an
Merci !
Testé en 3.2 SVN mutualisé et ça marche parfaitement bien !
Résultat :
Composed-By:SPIP 3.2.0 @ www.spip.net + http://spip32.local/sites/spip32.local/local/config.txt
#12
Mis à jour par RealET 🔸 il y a environ un an
- Fichier 3557_3.2.patch Voir ajouté
Et le patch en 3.2 'parce que c'est pas les mêmes lignes exactement)
#13 Mis à jour par Anonyme il y a environ un an
- Statut changé de En cours à Fermé
- % réalisé changé de 0 à 100
Appliqué par commit r23898.
#14
Mis à jour par b b il y a environ un an
- Version cible changé de 3.1 à 3.2
- Resolution mis à fixed