Produits Entr'ouvert » Passerelle

Je pensais plutôt à un log_requests_errors=False ?

Nicolas Roche a écrit :

Fait, mais ça renverra une 500, et continuera à faire planter le cron
(dans l'entête du ticket tu spécifiais : "plutôt répondre une 200").

J'ai la tête ailleurs, je ne comprends plus le code de Passerelle... c'est le self.requests qui provoque une exception en cas de 500 ?

Oui, simulé ci-dessous avec une URL qui ne répond pas.

  /home/nroche/src/passerelle/passerelle/apps/base_adresse/models.py(153)addresses()
-> result_response = self.requests.get('https://url-qui-ne-répond-pas/')
  /home/nroche/envs/publik-env-py3/lib/python3.9/site-packages/requests/sessions.py(555)get()
-> return self.request('GET', url, **kwargs)
  /home/nroche/src/passerelle/passerelle/utils/__init__.py(300)request()
-> response = super(Request, self).request(method, url, **kwargs)
  /home/nroche/envs/publik-env-py3/lib/python3.9/site-packages/requests/sessions.py(542)request()
-> resp = self.send(prep, **send_kwargs)
  /home/nroche/src/passerelle/passerelle/utils/__init__.py(317)send()
-> response = super(Request, self).send(request, **kwargs)
  /home/nroche/envs/publik-env-py3/lib/python3.9/site-packages/requests/sessions.py(655)send()
-> r = adapter.send(request, **kwargs)
> /home/nroche/envs/publik-env-py3/lib/python3.9/site-packages/requests/adapters.py(516)send()
-> raise ConnectionError(e, request=request)

Ok, donc il faut aussi faire un try/except sur ConnectionError lors du self.request.

Ca sera plus explicite que ce que tu proposais en premier :

        try:
            result_response = self.requests.get(url)
            result_response.raise_for_status()
        except RequestException as e:
            raise APIError('Bad response code from API: %s' % e)

qui "englobait" l'erreur de connexion dans une APIError "bad response" alors que non, y'a pas eu de réponse.

Je propose donc ce schéma :

        try:
            result_response = self.requests.get(url)
        except ConnectionError as e:
            raise APIError('Connection error: %s' % e)
        try;
            result_response.raise_for_status()
        except RequestException as e:
            raise APIError('Bad response code from API: %s' % e)

et donc remettre le log, en fait, c'était pas lui qu'on visait (mes excuses pour l'embrouille)

requests.get(.) ça peut renvoyer quand même n'importe quelle RequestException, pas juste ConnexionError (il me semble), après on peut évidemment spécialiser les except pour le cas ConnexionError.

Benjamin Dauvergne a écrit :

requests.get(.) ça peut renvoyer quand même n'importe quelle RequestException, pas juste ConnexionError (il me semble), après on peut évidemment spécialiser les except pour le cas ConnexionError.

T'as totalement raison, et d'ailleurs y'a des soucis de connexion qui ne sont pas des ConnectionError (timeout, certif.. voir https://github.com/psf/requests/blob/master/requests/exceptions.py).

Nicolas c'est donc bien RequestException qu'il faut dans le except et donc ta première proposition était la bonne, j'ai honte et il me faut des vacances :) Il faut cependant revoir le message de l'APIError car on couvre plus large que le seul cas de code non-200

Peut-être quelque chose comme :

        try:
            result_response = self.requests.get(url)
            result_response.raise_for_status()
        except RequestException as e:
            raise APIError('failed to get %s: %s' % (url, e))