entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 13 Releases Activity

Gérer les zeep.exceptions.Fault contenant des bytes and simplifier la gestion d'erreur autour des appels SOAP (#75649) #154

Merged
bdauvergne merged 4 commits from wip/75649-zeep-exceptions-Fault-est-toujou into main 2023-03-24 16:47:54 +01:00
Conversation 1 Commits 4 Files Changed 37 +131 -16811
bdauvergne commented 2023-03-21 12:25:31 +01:00
Owner
Copy Link
No description provided.
bdauvergne changed title from Gérer les zeep.exceptions.Fault contenant des bytes and simplifier la gestion d'erreur autour des appels SOAP (#75649) to WIP: Gérer les zeep.exceptions.Fault contenant des bytes and simplifier la gestion d'erreur autour des appels SOAP (#75649) 2023-03-21 12:25:50 +01:00
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from 12eb69066f to e7c27811d0 2023-03-21 18:38:32 +01:00 Compare
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from e7c27811d0 to a99658efd5 2023-03-21 18:58:27 +01:00 Compare
bdauvergne changed title from WIP: Gérer les zeep.exceptions.Fault contenant des bytes and simplifier la gestion d'erreur autour des appels SOAP (#75649) to Gérer les zeep.exceptions.Fault contenant des bytes and simplifier la gestion d'erreur autour des appels SOAP (#75649) 2023-03-22 18:56:39 +01:00
nroche approved these changes 2023-03-23 11:50:55 +01:00
nroche left a comment
Owner
Copy Link

Tu profites du fix sur Fault.detail avec 'bytes' pour simplifier tous les appels.
Ok pour moi.
(et merci d'avoir flingué le connecteur Maelis)

Il y a juste que dans ton premier commit tu as posé un test qui concerne le second.
Et que je trouve dommage qu'il n'y ait pas de test pour le bug qui a motivé ce ticket.

Tu profites du fix sur Fault.detail avec 'bytes' pour simplifier tous les appels. Ok pour moi. (et merci d'avoir flingué le connecteur Maelis) Il y a juste que dans ton premier commit tu as posé un test qui concerne le second. Et que je trouve dommage qu'il n'y ait pas de test pour le bug qui a motivé ce ticket.
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from a99658efd5 to 0c4650bd86 2023-03-23 15:29:31 +01:00 Compare
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from 0c4650bd86 to 17e18786ab 2023-03-23 15:31:59 +01:00 Compare
bdauvergne commented 2023-03-23 15:32:13 +01:00
Author
Owner
Copy Link

J'ai rajouté un test et déplacé le test mal placé.

J'ai rajouté un test et déplacé le test mal placé.
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from 17e18786ab to cf79039c1c 2023-03-24 15:44:26 +01:00 Compare
bdauvergne force-pushed wip/75649-zeep-exceptions-Fault-est-toujou from cf79039c1c to ba8c0cbd67 2023-03-24 16:06:01 +01:00 Compare
bdauvergne merged commit 57d2bb06a9 into main 2023-03-24 16:47:54 +01:00
bdauvergne deleted branch wip/75649-zeep-exceptions-Fault-est-toujou 2023-03-24 16:47:54 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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/passerelle#154
No description provided.
Delete Branch "wip/75649-zeep-exceptions-Fault-est-toujou"

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?