Projet

Général

Profil

Development #35395

throttling sur l'URL du code de suivi

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
14 août 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Comme w.c.s. #35386 et #35393.


Fichiers

Révisions associées

Révision 4836ed82 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

misc: add rate limiting to tracking code URLs (#35395)

Historique

#1

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

Rien de particulier ici, mais j'inclus le bout de #34405 qui n'était pas encore poussé dans la branche.

#2

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

J'ai l'impression que jamais le message de limite atteinte ne sera affiché à l'utilisateur ?

#3

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

Mon idée était que ça ne s'adressait de toute façon pas à l'usager légitime, qui ne passera jamais par là (cf w.c.s. où on passe le bout en anglais sans s'en soucier).

#4

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

Frédéric Péters a écrit :

Mon idée était que ça ne s'adressait de toute façon pas à l'usager légitime, qui ne passera jamais par là (cf w.c.s. où on passe le bout en anglais sans s'en soucier).

w.c.s je pensais que ça se voyait quelque part, j'ai raté le coche là bas aussi.

Je pensais bien à un usager légitime, qui est un agent dont l'IP est ratelimitée par trente collègues du call center qui jouent sur la même IP, et qui ne comprendra pas pourquoi il ne trouve pas un numéro de suivi dicté par l'usager qu'il a au bout du téléphone (parce que 1500/d). Je n'aimerai être ni cet agent qui va devenir un peu fou, ni la personne qui va devoir comprendre le soucis avec les explications aléatoires de l'agent, et l'impossibilité de reproduire.

#5

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

  • Statut changé de Solution proposée à En cours

Côté w.c.s. ça s'affichera quand même, de manière plutôt brute, à l'usager.

Je regarde pour ici faire les trucs de manière jolie. (au détriment du code…)

#6

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

Message explicite quand c'est fait depuis la cellule; quand ça arrive via la recherche c'est plus compliqué et je ne voulais pas me lancer dans plein d'infra pour ça, donc ça crée un faux résultat de recherche avec le message.

#7

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

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

Dans settings.py : # tracking code thorttling ← typo

Et ack dès que les tests verdissent.

#8

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4836ed82d84b1623fe2f1a31e71fb5a5054b8b37
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Aug 14 09:40:13 2019 +0200

    misc: add rate limiting to tracking code URLs (#35395)
#9

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF