Re: [SPIP Zone] nospam et ip du visiteur

2019-07-15 Par sujet Pierre KUHN
Bonjour,

J'ai trouvé dans le code des #INCLURE que j'ai replacer en  a écrit :

> Je n'ai pas la version 3 sur mes sites, je me replonge dans le code pour
> comprendre le problème.
>
> Merci pour la piste
>
> Le jeu. 27 juin 2019 à 15:20, RastaPopoulos  a
> écrit :
>
>> Le 27/06/2019 à 12:51, Pierre KUHN a écrit :
>> > Je suis sur un site avec le noisetier donc pas de #INCLURE en principe.
>>
>> Bé si justement.
>>
>> C'est un des problèmes que j'avais remonté à Éric quand il a fait la
>> refonte.
>>
>> Déjà moi j'arrive pas à voir dans quel cas ça serait indispensable
>> d'avoir une inclusion statique de noisettes, donc pour moi ça devrait
>> même pas être configurable, ça devrait toujours être dynamique. Mais en
>> plus, là on peut le choisir ET c'était statique par défaut !
>>
>> Moi je suis d'avis que :
>> 1) on devrait enlever le choix et ça devrait toujours être dynamique
>> 2) si vraiment on doit laisser le choix, ça DOIT être dynamique par
>> défaut, on ne doit rien avoir à rajouter pour ça, et c'est seulement
>> quand on veut du statique qu'on le dit
>>
>> Cf :
>>
>> https://zone.spip.net/trac/spip-zone/changeset/113519/spip-zone/_plugins_/noizetier/trunk
>>
>> Pour moi clairement ce choix ne va pas, ça doit absolument être
>> dynamique par défaut. Cédric a toujours expliqué qu'on devait toujours
>> mettre , et que seulement dans quelques cas hyper rare en ayant
>> bien réfléchi on décide de mettre #INCLURE : n-core et le noizetier
>> doivent suivre cette règle.
>>
>> --
>> RastaPopoulos
>>
>> 
>> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone
>>
>
>
> --
>
> Pierre KUHN
> http://www.voisins-spipeurs.net
>


-- 

Pierre KUHN
http://www.voisins-spipeurs.net

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-27 Par sujet Pierre KUHN
Je n'ai pas la version 3 sur mes sites, je me replonge dans le code pour
comprendre le problème.

Merci pour la piste

Le jeu. 27 juin 2019 à 15:20, RastaPopoulos  a
écrit :

> Le 27/06/2019 à 12:51, Pierre KUHN a écrit :
> > Je suis sur un site avec le noisetier donc pas de #INCLURE en principe.
>
> Bé si justement.
>
> C'est un des problèmes que j'avais remonté à Éric quand il a fait la
> refonte.
>
> Déjà moi j'arrive pas à voir dans quel cas ça serait indispensable
> d'avoir une inclusion statique de noisettes, donc pour moi ça devrait
> même pas être configurable, ça devrait toujours être dynamique. Mais en
> plus, là on peut le choisir ET c'était statique par défaut !
>
> Moi je suis d'avis que :
> 1) on devrait enlever le choix et ça devrait toujours être dynamique
> 2) si vraiment on doit laisser le choix, ça DOIT être dynamique par
> défaut, on ne doit rien avoir à rajouter pour ça, et c'est seulement
> quand on veut du statique qu'on le dit
>
> Cf :
>
> https://zone.spip.net/trac/spip-zone/changeset/113519/spip-zone/_plugins_/noizetier/trunk
>
> Pour moi clairement ce choix ne va pas, ça doit absolument être
> dynamique par défaut. Cédric a toujours expliqué qu'on devait toujours
> mettre , et que seulement dans quelques cas hyper rare en ayant
> bien réfléchi on décide de mettre #INCLURE : n-core et le noizetier
> doivent suivre cette règle.
>
> --
> RastaPopoulos
>
> 
> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone
>


-- 

Pierre KUHN
http://www.voisins-spipeurs.net

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-27 Par sujet RastaPopoulos
Le 27/06/2019 à 12:51, Pierre KUHN a écrit :
> Je suis sur un site avec le noisetier donc pas de #INCLURE en principe.

Bé si justement.

C'est un des problèmes que j'avais remonté à Éric quand il a fait la
refonte.

Déjà moi j'arrive pas à voir dans quel cas ça serait indispensable
d'avoir une inclusion statique de noisettes, donc pour moi ça devrait
même pas être configurable, ça devrait toujours être dynamique. Mais en
plus, là on peut le choisir ET c'était statique par défaut !

Moi je suis d'avis que :
1) on devrait enlever le choix et ça devrait toujours être dynamique
2) si vraiment on doit laisser le choix, ça DOIT être dynamique par
défaut, on ne doit rien avoir à rajouter pour ça, et c'est seulement
quand on veut du statique qu'on le dit

Cf :
https://zone.spip.net/trac/spip-zone/changeset/113519/spip-zone/_plugins_/noizetier/trunk

Pour moi clairement ce choix ne va pas, ça doit absolument être
dynamique par défaut. Cédric a toujours expliqué qu'on devait toujours
mettre , et que seulement dans quelques cas hyper rare en ayant
bien réfléchi on décide de mettre #INCLURE : n-core et le noizetier
doivent suivre cette règle.

-- 
RastaPopoulos


spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-27 Par sujet Pierre KUHN
Je rajoute $jeton et je refais mes tests.

Je suis sur un site avec le noisetier donc pas de #INCLURE en principe.

Je poursuis, merci.

Le jeu. 27 juin 2019 à 11:56, Cerdic  a écrit :

> Ajoute $jeton dans le log pour voir, mais je pense que tu as bien un
> jeton, il est juste plus valide parce qu’il a été mis en cache à cause d’un
> modele ou d’un #INCLURE, ou d’une mise en cache abusive en amont par ton
> load-balancer ou reverse-proxy
>
> Il est censé être dynamique est regénéré à chaque affichage du formulaire,
> et valide 1h, ce qui normalement est un temps suffisant pour qu’un
> utilisateur saisisse les infos dans le formulaire.
>
> --
> Cédric
> Le 27 juin 2019 à 11:48 +0200, Pierre KUHN , a
> écrit :
>
> Bonjour,
>
> En continuant des test, cela se produit aussi sans avoir de load-balancin.
> En fouillant dans le code j'ai dé-commenter
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/nospam_pipelines.php#L265
> cela me retourne "pas de jeton pour array('args' => array( 'form' =>
> 'formidable', 'args' => array( 0 => '12' ), ), 'data' => array(), )"
>
> De plus cela ne ce produit pas tout le temps du coup je ne vois pas
> comment remonter le problème plus haut.
>
> Des idées ?
>
> Merci.
>
> Le mar. 18 juin 2019 à 15:40, Bruno Bergot  a écrit :
>
> Hop,
>
> Le 18/06/2019 à 15:05, Pierre KUHN a écrit :
> > Bonjour,
> >
> > Je suis dans un contexte de serveur avec un load-balancing et l'ip qui
> > remonte est toujours l'ip d'un serveur du réseau.
> > Le jeton ne change pas si on ne recalcul pas la page, ce n'est pas un
> > problème de #INCLURE car le site est réaliser avec noisetier.
> >
> > Dans nospam, pour un visiteur sans compte nous utilisons $GLOBALS['ip']
> ici
> >
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/inc/nospam.php#L14
> > hors elle remonte une ip de serveur.
> >
>
> Amha le problème est du côté de la conf de ton loadbalancer qui devrait
> faire passer l'ip originale au serveur qui est derrière lui, à coup de
> X-Forwarded-For car SPIP prend déjà ça en charge cf :
>
>
> https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc_version.php#L297
>
> Code qui prend déjà en charge $_SERVER['REMOTE_ADDR'] comme tu peux le
> voir.
>
> Bref, le bug ne semble pas être dans SPIP mais dans la conf de ton
> serveur :)
>
> ++
> b_b
>
>
>
> --
>
> Pierre KUHN
> http://www.voisins-spipeurs.net
> 
> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone
>
>

-- 

Pierre KUHN
http://www.voisins-spipeurs.net

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-27 Par sujet Cerdic
Ajoute $jeton dans le log pour voir, mais je pense que tu as bien un jeton, il 
est juste plus valide parce qu’il a été mis en cache à cause d’un modele ou 
d’un #INCLURE, ou d’une mise en cache abusive en amont par ton load-balancer ou 
reverse-proxy

Il est censé être dynamique est regénéré à chaque affichage du formulaire, et 
valide 1h, ce qui normalement est un temps suffisant pour qu’un utilisateur 
saisisse les infos dans le formulaire.

--
Cédric
Le 27 juin 2019 à 11:48 +0200, Pierre KUHN , a écrit :
> Bonjour,
>
> En continuant des test, cela se produit aussi sans avoir de load-balancin.
> En fouillant dans le code j'ai dé-commenter 
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/nospam_pipelines.php#L265
> cela me retourne "pas de jeton pour array('args' => array( 'form' => 
> 'formidable', 'args' => array( 0 => '12' ), ), 'data' => array(), )"
>
> De plus cela ne ce produit pas tout le temps du coup je ne vois pas comment 
> remonter le problème plus haut.
>
> Des idées ?
>
> Merci.
>
> > Le mar. 18 juin 2019 à 15:40, Bruno Bergot  a écrit :
> > > Hop,
> > >
> > > Le 18/06/2019 à 15:05, Pierre KUHN a écrit :
> > > > Bonjour,
> > > >
> > > > Je suis dans un contexte de serveur avec un load-balancing et l'ip qui
> > > > remonte est toujours l'ip d'un serveur du réseau.
> > > > Le jeton ne change pas si on ne recalcul pas la page, ce n'est pas un
> > > > problème de #INCLURE car le site est réaliser avec noisetier.
> > > >
> > > > Dans nospam, pour un visiteur sans compte nous utilisons $GLOBALS['ip'] 
> > > > ici
> > > > https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/inc/nospam.php#L14
> > > > hors elle remonte une ip de serveur.
> > > >
> > >
> > > Amha le problème est du côté de la conf de ton loadbalancer qui devrait
> > > faire passer l'ip originale au serveur qui est derrière lui, à coup de
> > > X-Forwarded-For car SPIP prend déjà ça en charge cf :
> > >
> > > https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc_version.php#L297
> > >
> > > Code qui prend déjà en charge $_SERVER['REMOTE_ADDR'] comme tu peux le 
> > > voir.
> > >
> > > Bref, le bug ne semble pas être dans SPIP mais dans la conf de ton
> > > serveur :)
> > >
> > > ++
> > > b_b
>
>
> --
>
> Pierre KUHN
> http://www.voisins-spipeurs.net
> 
> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-27 Par sujet Pierre KUHN
Bonjour,

En continuant des test, cela se produit aussi sans avoir de load-balancin.
En fouillant dans le code j'ai dé-commenter
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/nospam_pipelines.php#L265
cela me retourne "pas de jeton pour array('args' => array( 'form' =>
'formidable', 'args' => array( 0 => '12' ), ), 'data' => array(), )"

De plus cela ne ce produit pas tout le temps du coup je ne vois pas comment
remonter le problème plus haut.

Des idées ?

Merci.

Le mar. 18 juin 2019 à 15:40, Bruno Bergot  a écrit :

> Hop,
>
> Le 18/06/2019 à 15:05, Pierre KUHN a écrit :
> > Bonjour,
> >
> > Je suis dans un contexte de serveur avec un load-balancing et l'ip qui
> > remonte est toujours l'ip d'un serveur du réseau.
> > Le jeton ne change pas si on ne recalcul pas la page, ce n'est pas un
> > problème de #INCLURE car le site est réaliser avec noisetier.
> >
> > Dans nospam, pour un visiteur sans compte nous utilisons $GLOBALS['ip']
> ici
> >
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/inc/nospam.php#L14
> > hors elle remonte une ip de serveur.
> >
>
> Amha le problème est du côté de la conf de ton loadbalancer qui devrait
> faire passer l'ip originale au serveur qui est derrière lui, à coup de
> X-Forwarded-For car SPIP prend déjà ça en charge cf :
>
>
> https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc_version.php#L297
>
> Code qui prend déjà en charge $_SERVER['REMOTE_ADDR'] comme tu peux le
> voir.
>
> Bref, le bug ne semble pas être dans SPIP mais dans la conf de ton
> serveur :)
>
> ++
> b_b
>


-- 

Pierre KUHN
http://www.voisins-spipeurs.net

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] nospam et ip du visiteur

2019-06-18 Par sujet Bruno Bergot

Hop,

Le 18/06/2019 à 15:05, Pierre KUHN a écrit :

Bonjour,

Je suis dans un contexte de serveur avec un load-balancing et l'ip qui
remonte est toujours l'ip d'un serveur du réseau.
Le jeton ne change pas si on ne recalcul pas la page, ce n'est pas un
problème de #INCLURE car le site est réaliser avec noisetier.

Dans nospam, pour un visiteur sans compte nous utilisons $GLOBALS['ip'] ici
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/nospam/inc/nospam.php#L14
hors elle remonte une ip de serveur.



Amha le problème est du côté de la conf de ton loadbalancer qui devrait 
faire passer l'ip originale au serveur qui est derrière lui, à coup de 
X-Forwarded-For car SPIP prend déjà ça en charge cf :


https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc_version.php#L297

Code qui prend déjà en charge $_SERVER['REMOTE_ADDR'] comme tu peux le voir.

Bref, le bug ne semble pas être dans SPIP mais dans la conf de ton 
serveur :)


++
b_b

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone