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)
Associated revisions
Fix #3557 : simplifier l'entête compsed-by en y indiquant l'url du fichier local/config.txt
History
#1
Updated by RealET 🔸 over 5 years ago
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
Updated by b b over 5 years ago
- Subject changed from http://stats.spip.net/ faussées to Header composed-by tronqué
#3
Updated by RealET 🔸 almost 5 years ago
Autre outil impacté par cette limitation : http://blog.nursit.net/Administrer-les-plugins-d-un-site.html
#4
Updated by RealET 🔸 almost 5 years ago
http://zone.spip.org/trac/spip-zone/changeset/97960 permet de chercher à prendre en compte les sites en mutualisation facile.
#5
Updated by RealET 🔸 about 4 years ago
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
Updated by b b about 4 years ago
- Target version set to 3.1
#7
Updated by b b about 3 years ago
- Status changed from Nouveau to 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
Updated by RealET 🔸 about 3 years ago
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
Updated by b b about 3 years ago
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
Updated by b b about 3 years ago
- File 3557.patch View added
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
Updated by RealET 🔸 about 3 years ago
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
Updated by RealET 🔸 about 3 years ago
- File 3557_3.2.patch View added
Et le patch en 3.2 'parce que c'est pas les mêmes lignes exactement)
#13 Updated by Anonymous about 3 years ago
- Status changed from En cours to Fermé
- % Done changed from 0 to 100
Appliqué par commit r23898.
#14
Updated by b b about 3 years ago
- Target version changed from 3.1 to 3.2
- Resolution set to fixed