[SPIP Zone] [Cachelab] fbclid -> 1 cache par visiteur unique

Archives Zone
1

En effet, on peut les virer dans un fichier .htaccess comme ça

# attaque des zombies facebook
RewriteCond %{QUERY_STRING} ^(.*)&?fbclid=[^&]+&?(.*)$ [NC]
RewriteRule ^(.*)$ /$1?%1%2 [R=301,L]

Dans les stats c'est géré depuis récemment pour ne pas compter des
refferers différents.
https://zone.spip.net/trac/spip-zone/changeset/112346/spip-zone

Merci,

Le mieux ne serait-il pas de faire la même chose pour le cache et les
stats ?

Mise en commun des paramètres d'url perturbants ?

2

Le 04/01/2019 à 17:21, eric a écrit :

Le mieux ne serait-il pas de faire la même chose pour le cache et les
stats ?
Mise en commun des paramètres d'url perturbants ?

Ces paramètres parasites sont déjà pris en compte pour ne pas perturber la génération des caches :
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/public/assembler.php#L26
utilisé dans calculer_contexte en ligne 192
Peut être manque t il amp ou amp=1
mais c'est extensible via define dans un fichier d'options.

C'est une liste légèrement différente qui est utilisée dans nettoyer_uri_var
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/utils.php#L639
qui sert à d'autres usages, mais peut être faudrait il unifier.

JL

3

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/publ
ic/assembler.php#L26
utilisé dans calculer_contexte en ligne 192
Peut être manque t il amp ou amp=1
mais c'est extensible via define dans un fichier d'options.

C'est une liste légèrement différente qui est utilisée dans
nettoyer_uri_var
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/
utils.php#L639
qui sert à d'autres usages, mais peut être faudrait il unifier.

Merci JL,
je teste.

4

Le 04/01/2019 à 20:05, eric a écrit :

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/publ
ic/assembler.php#L26
utilisé dans calculer_contexte en ligne 192
Peut être manque t il amp ou amp=1
mais c'est extensible via define dans un fichier d'options.

C'est une liste légèrement différente qui est utilisée dans
nettoyer_uri_var
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/
utils.php#L639
qui sert à d'autres usages, mais peut être faudrait il unifier.

Merci JL,
je teste.

Il se peut qu'il y ait un problème avec ce test car je constate avec XRay qu'il y a des fbclid dans les contextes des caches.

JLuc

5

Le 04/01/2019 à 22:19, JLuc a écrit :

Le 04/01/2019 à 20:05, eric a écrit :

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/publ
ic/assembler.php#L26
utilisé dans calculer_contexte en ligne 192
Peut être manque t il amp ou amp=1
mais c'est extensible via define dans un fichier d'options.

C'est une liste légèrement différente qui est utilisée dans
nettoyer_uri_var
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/
utils.php#L639
qui sert à d'autres usages, mais peut être faudrait il unifier.

Merci JL,
je teste.

Il se peut qu'il y ait un problème avec ce test car je constate avec XRay qu'il y a des fbclid dans les contextes des caches.

Ah non, ouf !
Le site sur lequel je testais est en 3.1 et la REGEXP ne testait pas encore fbclid à cette époque.
Sur un site à jour, il n'y a pas de fbclid en variables d'environnement dans les contextes,
mais j'en trouve quelques uns indirectement dans les valeurs de actions des formulaires :

Par exemple pour un formulaire du plugin notations :
[action] => /article.php?id_article=407&fbclid=IwAR0cticoD1HuOkKAdrKzQsRrekFeKzl90w_iL2Ru48XncnDoZ0yn79sTAuk
Peut être parce que ce serait filtré par nettoyer_uri_var qui est en retard par rapport à la REGEXP ?

JL

6

C'est tester avec cachelab/memoization et SPIP 3.2.1

dans mes_options.php :

#ne pas prendre en compte les paramètres suivants pour
création/récupération du cache
define('_CONTEXTE_IGNORE_VARIABLES',
"/(^var_|^PHPSESSID$|^fbclid$|^utm_)/");

C'est tester : plus de création de caches différents pour une même page
et pour chaque paramètre fbclid passé lors d'une visite en provenance
de Facebook.

Merci,

Eric

PS : je suis le dépot Spip github : cela est à appliquer également pour
3.2.1?
https://github.com/spip/SPIP/pull/33/commits/eefc863907ba9002aec797b376
819a8f789dc74d