entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

sms: do not record errors for application-level errors (#65330) #466

Merged
fpeters merged 1 commits from wip/65330-sms-error-log into main 2023-07-14 09:04:07 +02:00
Conversation 0 Commits 1 Files Changed 1 +4 -2
fpeters commented 2023-07-09 10:35:54 +02:00
Owner
Copy Link

Choix de ne plus enregistrer ce qui correspond à "APIError" côté passerelle (par exemple si un numéro fixe est reçu); par contre on continue à logguer pour des erreurs plus bas niveau (genre une erreur 404 qui correspondrait à une mauvaise url vers le connecteur).

Il y a un entre-deux amené par la validation de schéma qui produira une erreur 400, ça fait que si on envoie vraiment n'importe quoi (exemple la chaine "None") ça va être enregistré comme erreur. Ma position là-dessus serait le retrait de la validation par schéma de Passerelle, que ça passe dans clean_numbers/authorize_numbers et produise également une APIError 200).

Choix de ne plus enregistrer ce qui correspond à "APIError" côté passerelle (par exemple si un numéro fixe est reçu); par contre on continue à logguer pour des erreurs plus bas niveau (genre une erreur 404 qui correspondrait à une mauvaise url vers le connecteur). Il y a un entre-deux amené par la validation de schéma qui produira une erreur 400, ça fait que si on envoie vraiment n'importe quoi (exemple la chaine "None") ça va être enregistré comme erreur. Ma position là-dessus serait le retrait de la validation par schéma de Passerelle, que ça passe dans clean_numbers/authorize_numbers et produise également une APIError 200).
fpeters added 1 commit 2023-07-09 10:35:55 +02:00
fpeters changed title from WIP: sms: do not record errors for application-level errors (#65330) to sms: do not record errors for application-level errors (#65330) 2023-07-09 11:41:56 +02:00
lguerin approved these changes 2023-07-10 09:14:56 +02:00
fpeters merged commit b28aaaa033 into main 2023-07-14 09:04:07 +02:00
fpeters deleted branch wip/65330-sms-error-log 2023-07-14 09:04:07 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#466
No description provided.
Delete Branch "wip/65330-sms-error-log"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?