Anomalie #3557

Header composed-by tronqué

Ajouté par RealET 🔶 il y a plus de 2 ans. Mis à jour il y a 4 mois.

Statut:FerméDébut:27/09/2015
Priorité:BasEchéance:
Assigné à:-% réalisé:

100%

Catégorie:code généré
Version cible:3.2
Resolution:fixed Navigateur:

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)

3557.patch Magnifier (1,369 ko) b b, 25/12/2017 18:26

3557_3.2.patch Magnifier (1,41 ko) RealET 🔶, 25/12/2017 19:01

Révisions associées

Révision 23898
Ajouté par brunobergot@gmail.com il y a 4 mois

Fix #3557 : simplifier l'entête compsed-by en y indiquant l'url du fichier local/config.txt

Révision 23899
Ajouté par brunobergot@gmail.com il y a 4 mois

report de r23898

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 2 ans

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 2 ans

  • Sujet changé de http://stats.spip.net/ faussées à Header composed-by tronqué

#3 Mis à jour par RealET 🔶 il y a environ 2 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 2 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 plus d'un an

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 plus d'un an

  • Version cible mis à 3.1

#7 Mis à jour par b b il y a 5 mois

  • 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 5 mois

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 :

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

qui permettrait :
  • 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 5 mois

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 5 mois

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 5 mois

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 5 mois

Et le patch en 3.2 'parce que c'est pas les mêmes lignes exactement)

#13 Mis à jour par Anonyme il y a 4 mois

  • 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 4 mois

  • Version cible changé de 3.1 à 3.2
  • Resolution mis à fixed

Formats disponibles : Atom PDF

Ajouter une image à partir du presse-papier (Taille maximale: 1,25 Mo)