Project

General

Profile

Anomalie #3557

Header composed-by tronqué

Added by RealET 🔸 over 5 years ago. Updated about 3 years ago.

Status:
Fermé
Priority:
Bas
Assignee:
-
Category:
code généré
Target version:
3.2
Start date:
09/27/2015
Due date:
% Done:

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 View (1.37 KB) b b, 12/25/2017 06:26 PM

3557_3.2.patch View (1.41 KB) RealET 🔸, 12/25/2017 07:01 PM

Associated revisions

Revision 23898 (diff)
Added by brunobergot@gmail.com about 3 years ago

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

Revision 23899 (diff)
Added by brunobergot@gmail.com about 3 years ago

report de r23898

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 :

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

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

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

Also available in: Atom PDF