Bug #72597
le ratelimiting sur l’envoi de SMS est bogué
Start date:
19 December 2022
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
La fréquence d’un SMS par heure est trop restrictive.
Plus problématique encore, la clé-accesseur pour le comptage est erronée : 'post:sms' alors que ce devrait être 'post:phone' (qui veut dire qu’on va chercher le contenu de request.POST['phone']).
Files
Related issues
Associated revisions
History
Updated by Paul Marillonnet almost 2 years ago
- Status changed from Nouveau to En cours
- Assignee set to Paul Marillonnet
Updated by Paul Marillonnet almost 2 years ago
- Related to Développement #49212: Création de compte avec un numéro de téléphone mobile added
Updated by Paul Marillonnet almost 2 years ago
- File 0001-views-fix-sms-registration-phone-number-ratelimit-ke.patch 0001-views-fix-sms-registration-phone-number-ratelimit-ke.patch added
- Status changed from En cours to Solution proposée
- Patch proposed changed from No to Yes
Updated by A. B. almost 2 years ago
- Status changed from Solution proposée to Solution validée
Très clair ce test !
Updated by Paul Marillonnet almost 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
Agate Berriot a écrit :
Très clair ce test !
J’avoue j’ai copié sur le test déjà existant qui fait la même chose mais pour le ratelimiting sur la création de compte par email :)
commit 38e12e840c1bb749e6500f4a01534781c05bead3
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Mon Dec 19 15:10:49 2022 +0100
views: fix sms-registration phone-number ratelimit key (#72597)
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
views: fix sms-registration phone-number ratelimit key (#72597)