Projet

Général

Profil

Bug #49260

éviter de générer un brouillon sur un appel par python-requests (et autres logiciels robots)

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 décembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision ada843ca (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

misc: do not save initial draft for requests from applications (#49260)

Historique

#2

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 ?

#3

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.

#4

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)
#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Assigné à mis à Frédéric Péters
#7

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

#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

#9

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.

#10

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.

#11

Mis à jour par Frédéric Péters il y a plus de 3 ans

Oui il est dans la branche et pas dans le patch attaché.

#12

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#13

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)
#14

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

Formats disponibles : Atom PDF