Bug #49260
éviter de générer un brouillon sur un appel par python-requests (et autres logiciels robots)
0%
Description
Typiquement, on a dans un combo une cellule lien avec une URL vers {{wcs}}/formulaire/?foo=bar
Lorsque combo va vérifier la cellule, il lance une requete vers wcs... qui va créer un brouillon.
Sur certains sites, cela amène à la création de (dizaines de) milliers de brouillons.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
Plutôt côté w.c.s. zapper la création de brouillon en cas d'appel avec python-requests en user agent ?
Mis à jour par Thomas Noël il y a plus de 3 ans
- Projet changé de Combo à w.c.s.
Frédéric Péters a écrit :
Plutôt côté w.c.s. zapper la création de brouillon en cas d'appel avec python-requests en user agent ?
Je n'osais pas. Et curl/wget, aussi, sans doute.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Sujet changé de génération de brouillons sur wcs par le vérificateur automatique d'URL dans les cellule à éviter de générer un brouillon sur un appel par python-requests (et autres logiciels robots)
- Description mis à jour (diff)
Mis à jour par Stéphane Laget il y a plus de 3 ans
et si l'usager clique sur "reprendre un brouillon", il perd l'info transmise dans l'url
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-misc-do-not-save-initial-draft-for-requests-from-app.patch 0001-misc-do-not-save-initial-draft-for-requests-from-app.patch ajouté
Je n'ai pas voulu mêler les applications (python-requests) et les bots (et j'ai pour l'occasion sorti Wget des bots); mon idée étant qu'on peut être agressif sur les bots, et qu'on ne veut pas nécessairement l'être sur les applications (genre ça me ferait chier d'avoir une 403 sur un curl de debug).
Mis à jour par Frédéric Péters il y a plus de 3 ans
et si l'usager clique sur "reprendre un brouillon", il perd l'info transmise dans l'url
Pas de rapport.
Mis à jour par Thomas Noël il y a plus de 3 ans
Frédéric Péters a écrit :
et j'ai pour l'occasion sorti Wget des bots
C'est peut-être un bout du patch qui manque (le retrait de Wget de data/webbots) ou alors je rate un truc ?
Le reste m'a l'air tout bon.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-misc-do-not-save-initial-draft-for-requests-from-app.patch 0001-misc-do-not-save-initial-draft-for-requests-from-app.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Oui il est dans la branche et pas dans le patch attaché.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ada843ca5550c20489aa25dc03e959244e2a1711 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Dec 8 16:34:34 2020 +0100 misc: do not save initial draft for requests from applications (#49260)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: do not save initial draft for requests from applications (#49260)