Produits Entr'ouvert » Passerelle

Ce patch est la seule idée que j'ai : en cas de pépin autre qu'une erreur SOAP normale (500), on créée une pseudo-erreur SOAP...

• ça ne donne pas le code d'erreur tel quel (et donc ce n'est pas ce qui est demandé) :

{'err': 1, 'err_desc': "error: Server raised fault: 'http transport error 503'", 'data': None, 'err_class': u'passerelle.utils.jsonresponse.APIError'}

• et surtout le test ajouté montre aussi que Greco ne passe par en py3 :
  

    File "/home/thomas/dev/publik/src/passerelle/passerelle/contrib/greco/models.py", line 125, in send
      request.message = request.message.replace("contentType", "xm:contentType")
  TypeError: a bytes-like object is required, not 'str'
  

et donc c'est plutôt une piste qu'une solution. Qu'en pensez-vous ?

Voici donc une solution simple, APIError étant bien "remontée" tranquillement au travers de suds.

Mini patch 0001 pour que le test passe également en py3 (sans garantir que le connecteur dans son ensemble tourne en py3, la couverture est minimale, autre sujet, autre ticket, autre patch)

Benjamin Dauvergne a écrit :

Frédéric Péters a écrit :

Une raison pour ignorer l'erreur 500 ?

Ça peut contenir un vrai message (soap:Fault), mais doit y avoir moyen de différencier (genre juste chercher "Fault" dans le contenu de la réponse).

Ouaip, je vois mal comment ça pourrait arriver (une 500 c'est l'appli qui la produit, en général l'appli c'est le front SOAP) mais on ne sait jamais. J'ai donc ajouté le test proposé par Benjamin.

Ajouté aussi le status_code dans le message de l'APIError :

+                if resp.status_code >= 400 and resp.status_code != 500:
+                    raise APIError('HTTP Transport Error %s' % resp.status_code,
+                                   err_code='transport-error-%s' % resp.status_code)
+                elif resp.status_code == 500 and 'Fault' not in resp.content:
+                    raise APIError('Error 500, not a SOAP Fault',
+                                   err_code='transport-error-500')

C'est effectivement bien vu, j'ai envoyé sur la branche cet amendement :

@@ -183,7 +183,7 @@ class Greco(BaseResource):
                 if resp.status_code >= 400 and resp.status_code != 500:
                     raise APIError('HTTP Transport Error %s' % resp.status_code,
                                    err_code='transport-error-%s' % resp.status_code)
-                elif resp.status_code == 500 and 'Fault' not in resp.content:
+                elif resp.status_code == 500 and b'Fault' not in resp.content:
                     raise APIError('Error 500, not a SOAP Fault',
                                    err_code='transport-error-500')
                 return Reply(resp.status_code, resp.headers, resp.content)

Poussé avec :

-                request.message = request.message.replace("contentType", "xm:contentType")
+                request.message = request.message.replace(b"contentType", b"xm:contentType")

à la place du force_text, merci Benjamin.

commit 2069bcf80f212bc74a71521f00365dca2df7b80e
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Mar 25 01:51:18 2020 +0100

    greco: replace bytes in request.message

commit 54c64ec747f191ee485c15627e3e5ea33f00f238
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Mar 24 15:50:58 2020 +0100

    greco: raise APIError on http transport error (#40974)