Produits Entr'ouvert » Passerelle

• json_schema pour ne pas avoir à taper post={'request_body': {'application/json': ...}}
• response_schema pour pouvoir décrire le schéma en sortie (ça m'étonnait qu'on ait pas ça encore)

response_schema pour pouvoir décrire le schéma en sortie (ça m'étonnait qu'on ait pas ça encore)

#21774 pour cette partie ?

Pour les autres commits génériques, tu peux aussi leur donner des tickets ?

Pas grand chose à relire, le connecteur ne sert quasiment qu'à documenter les schémas et poser deux entêtes.

getattr(self.requests, method.lower())(full_path, *args, **kwargs) ça pourrait s'écrire plus joliment (et comme dans d'autres connecteurs), self.requests.request(method, full_path, *args, **kwargs).

raise requests.RequestException quand on ne reçoit pas du JSON, je crois que l'usage est tout de même de lever APIError.

with open('/tmp/rsa.log', 'a'), du debug qui traine.

Je trouve que la gymnastique entre les méthodes request et request_err est bizarre, ça serait vraiment mieux d'avoir juste une méthode request. D'ailleurs, quel est la raison de ces deux méthodes ? Le endpoint 'check' qui ne renvoie pas 'err': 0 ?

Valentin Deniaud a écrit :

getattr(self.requests, method.lower())(full_path, *args, **kwargs) ça pourrait s'écrire plus joliment (et comme dans d'autres connecteurs), self.requests.request(method, full_path, *args, **kwargs).

Ok.

raise requests.RequestException quand on ne reçoit pas du JSON, je crois que l'usage est tout de même de lever APIError.

C'est plus compliqué que ça mais ok.

with open('/tmp/rsa.log', 'a'), du debug qui traine.

Viré.

Je trouve que la gymnastique entre les méthodes request et request_err est bizarre, ça serait vraiment mieux d'avoir juste une méthode request. D'ailleurs, quel est la raison de ces deux méthodes ? Le endpoint 'check' qui ne renvoie pas 'err': 0 ?

Ouaip, j'ai renommé request en request_raw et request_err en request. C'est pas bien important mais la plus appelée a un nom plus court maintenant.

Branche à jour.