Development #54607
crash API Particulier quand numeroAllocataire=None
0%
Description
À cause de traces reçues je pensais API Particulier down mais en fait non il juste plante sur un appel avec numeroAllocataire=None
On peut chercher dans les démarches concernées les moments avec ces appels erronés mais on pourrait juste intercepter cette valeur dans le connecteur et répondre en erreur, sans envoyer ça à API Particulier.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Fichier 0001-api_particulier-validate-numero_allocataire-54607.patch 0001-api_particulier-validate-numero_allocataire-54607.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est bien défini ce que c'est un numéro allocataire, c'est un nombre à 7 chiffres, donc autant valider ça.
Mis à jour par Benjamin Dauvergne il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Il faut plutôt utiliser v.isdecimal() and v.isascii()
, car '²'.isdigit() is True
par exemple, le .isascii()
c'est plus de la coquetterie mais il y a des nombreux décimaux hors plage ASCII aussi et sinon il y a passerelle.utils.validation.is_number qui est sensé gérer et cacher tout ça.
Mis à jour par Valentin Deniaud il y a presque 3 ans
Benjamin Dauvergne a écrit :
il y a passerelle.utils.validation.is_number qui est sensé gérer et cacher tout ça.
Merci, fait sur la branche, je pousserai ça vendredi.
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 77f3fe355155a025ab817d4d4d84e98052190074 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Jun 8 10:59:12 2021 +0200 api_particulier: validate numero_allocataire (#54607)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api_particulier: validate numero_allocataire (#54607)