Produits Entr'ouvert » Passerelle

Est-il rebasé sur master ?

Oui

Patch à revoir suite au refactoring du connecteur solis (#5188)

Serghei Mihai a écrit :

Patch à revoir suite au refactoring du connecteur solis (#5188)

Nope ; Josué a écrit un nouveau connecteur Solis. L'ancien ne doit pas bouger (en prod sur une vieille CentOS au CG14).

Ah. Donc on n'est même pas sûr d'avoir requests.
Alors je vire les modifications du connecteur solis du patch

mock_response.json.return_value = {'ok'}

Quelle chaine json produirait ça ?

Il y a un changement d'API si on passe de :

            response = e.read()
            response = json.loads(response)
            return {'success': False, 'error': response}

à

            raise APIError('ClicRDV error: %s' % e)

On perd de l'info renvoyée par ClicRDV, ici le message d'erreur. Or c'est un message qui est utilisé dans les workflows, en cas de pépin.

Par ailleurs, les appels actuels à ClicRDV (Vincennes) utilisent la valeur success, et ici il n'y en aura pas.

Pour moi, requests d'abord, mais il faut conserver l'API actuelle.

Si on veut changer l'API, il faut étudier les usages actuels et en prévoir la modification.

Ça devient quand même un peu ridicule comme fonction :

     def get_json(self, uri):
-        req = self.get_request(uri)
-        return json.load(urllib2.urlopen(req))
+        response = self.request(uri)
+        return response.json()

C'est quand même dommage de perdre les informations précises fournies par clicrdv :

-            response = e.read()
-            response = json.loads(response)
-            return {'success': False, 'error': response}
+            raise APIError('ClicRDV error: %s' % e)

Plus loin on essaie davantage :

+            r.raise_for_status()
+        except requests.exceptions.HTTPError as e:
             try:
-                error = json.load(e.fp)[0].get('error')
+                error = json.loads(str(e)).get('error')
             except:
                 error = 'Unknown error (Passerelle)'

Mais ça ne risque pas de marcher, chez moi j'ai :

>>> str(e)
'500 Server Error: INTERNAL SERVER ERROR for url: http://passerelle.127.0.0.1.xip.io:8006/clicrdv/test/interventionsets'

et je pense que tu veux plutôt :

>>> r.content
'{"err_class": "urllib2.URLError", "err_desc": "<urlopen error [Errno -2] Name or service not known>", "data": null, "err": 1}'

Et derrière à nouveau quand même l'information structurée qu'on aurait pu récupérer est gaspillée avec :

+            raise APIError('ClicRDV error: %s' % error)

Bien sûr il ne s'agit pas de taper l'affectation dans le try/except, uniquement le raise_for_status.

Ligne 145 :

return {'success': False, 'error': response.get('error')}

Alors qu'avant tu ne supposais pas qu'il y avait une entrée 'error' dans le JSON, c'est normal ?