atal: handle error when uploading big file (#76884) #229

Merged
ecazenave merged 1 commits from wip/76884-atal-soap-fault into main 2023-05-03 11:37:33 +02:00
Owner

J'attrape l'exception dans le endpoint pour aller au plus simple (j'aurais pu attraper un niveau plus bas mais ça demanderait petit refactoring, que je m'évite parce que ce connecteur est obsolète ... SOAP plus support sur les dernières versions de ATAL).

J'attrape l'exception dans le endpoint pour aller au plus simple (j'aurais pu attraper un niveau plus bas mais ça demanderait petit refactoring, que je m'évite parce que ce connecteur est obsolète ... SOAP plus support sur les dernières versions de ATAL).
ecazenave force-pushed wip/76884-atal-soap-fault from 38563bb009 to d798fcd749 2023-04-24 14:40:10 +02:00 Compare
bdauvergne requested changes 2023-05-02 10:46:22 +02:00
bdauvergne left a comment
Owner

Il faut juste ajouter api_error=True quand tu instancies ton soap_client et tout ça est déjà fait (et ça gèrera aussi les erreurs de transport).

self.soap_client(wsdl_url=wsdl_url) -> self.soap_client(wsdl_url=wsdl_url, api_error=True)

Il faut juste ajouter api_error=True quand tu instancies ton soap_client et tout ça est déjà fait (et ça gèrera aussi les erreurs de transport). self.soap_client(wsdl_url=wsdl_url) -> self.soap_client(wsdl_url=wsdl_url, api_error=True)
Author
Owner

Je viens de passer 20 minutes à essayer de modifier le test, et je vais d'erreur en erreur, c'est déjà trop de temps passé pour un connecteur mourant.

Je viens de passer 20 minutes à essayer de modifier le test, et je vais d'erreur en erreur, c'est déjà trop de temps passé pour un connecteur mourant.
bdauvergne approved these changes 2023-05-03 11:33:02 +02:00
bdauvergne left a comment
Owner

Ok.

Ok.
ecazenave merged commit 822a0d83b4 into main 2023-05-03 11:37:33 +02:00
ecazenave deleted branch wip/76884-atal-soap-fault 2023-05-03 11:37:33 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#229
No description provided.