Development #67763
base adresse : limiter le "q" envoyé à 200 caractères
0%
Description
https://sentry.entrouvert.org/entrouvert/nfrance/issues/62632/
GET https://passerelle.eservices.toulouse-metropole.fr/pfs/gtm/search/?q=NOMS+des+bornes+%3A++HEURE+et+PANNE+%3A+HEURE+d%80%99appel+au+service+de+maintenance+SPIE+%3A+N%B0+d%80%99INTERVENTION+%3A+HEURE+d%80%99ARRIV%89E+du+technicien+%3A+Le+technicien+nous+informe+%28+r%A9paration%29++HEURE+de+D%89PART+du+technicien+%28+en+pr%A9cisant+si+les+bornes+sont+en+position+haute%29++&limit=5&citycode=31555&lat=43.6042618680962&lon=1.44332885742188 (=> 413)
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a presque 2 ans
- Projet changé de Suivi des traces à Passerelle
413 Payload Too Large
Et pour cause :
...q=NOMS des bornes : HEURE et PANNE : HEURE d’appel au service de maintenance SPIE : N° d’INTERVENTION : HEURE d’ARRIVÉE du technicien : Le technicien nous informe ( réparation) HEURE de DÉPART du technicien ( en précisant si les bornes sont en position haute)
C'est sur la ban de toulouse, mais on peut reproduire sur la ban nationale :
https://api-adresse.data.gouv.fr/search/?q=12345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890X&citycode=pas_pris_en_compte_dans_le_decompte
Query too long, 244 chars, limit is 200
Peut-être qu'on pourrait limiter la taille du paramètre q à 200 dans le connecteur ?
Mis à jour par Thomas Noël il y a presque 2 ans
- Sujet changé de GET https://passerelle.eservices.toulouse-metropole.fr/pfs/gtm/search/?q=NOMS+des+bornes+%3A++HEU... à base adresse : limiter le "q" envoyé à 200 caractères
Oui, il faut.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Fichier 0001-base_adresse-truncate-q-to-200-characters-67763.patch 0001-base_adresse-truncate-q-to-200-characters-67763.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Ack
(Ça ne sert à rien de retourner une erreur si l'on précise d'avantage la chaîne recherchée, je n'y avais pas pensé).
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
Nicolas Roche a écrit :
Ack
(Ça ne sert à rien de retourner une erreur si l'on précise d'avantage la chaîne recherchée, je n'y avais pas pensé).
Garbage in, garbage out.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7a04d9253f9f87fdbb31e2df47bbd2756f9eaa54 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Jul 28 11:31:29 2022 +0200 base_adresse: truncate q to 200 characters (#67763)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
base_adresse: truncate q to 200 characters (#67763)