Projet

Général

Profil

Anomalie #3557

Header composed-by tronqué

Ajouté par RealET 🔸 il y a environ 3 ans. Mis à jour il y a 10 mois.

Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Catégorie:
code généré
Version cible:
Début:
27/09/2015
Echéance:
% réalisé:

100%

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 Voir (1,37 ko) b b, 25/12/2017 18:26

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

Révisions associées

Révision 23898 (diff)
Ajouté par brunobergot@gmail.com il y a 10 mois

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

Révision 23899 (diff)
Ajouté par brunobergot@gmail.com il y a 10 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 environ 3 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 environ 3 ans

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

#3 Mis à jour par RealET 🔸 il y a plus de 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 plus de 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 presque 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 presque 2 ans

  • Version cible mis à 3.1

#7 Mis à jour par b b il y a 11 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 11 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 11 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 11 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 11 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 11 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 10 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 10 mois

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

Formats disponibles : Atom PDF