Bug #16472
Les exceptions httplib.HTTPException ne sont pas gérées dans _http_request
0%
Description
Or ça arrive #16470 (httplib.BadStatusLine).
Fichiers
Demandes liées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 7 ans
- Fichier 0001-handle-httplib.HTTPException-in-_http_request-16472.patch 0001-handle-httplib.HTTPException-in-_http_request-16472.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 7 ans
Manque un test, et le format du message d'erreur est différent de celui juste au-dessus.
-'error during HTTP exchange : %s' % err +'error during HTTP exchange (%s)' % err
Mis à jour par Benjamin Dauvergne il y a presque 7 ans
Frédéric Péters a écrit :
Manque un test, et le format du message d'erreur est différent de celui juste au-dessus.
[...]
Ok pour le test, pour le format ben c'est le même format que deux lignes encore plus haut, donc bon.
Mis à jour par Frédéric Péters il y a presque 7 ans
Ok pour le test, pour le format ben c'est le même format que deux lignes encore plus haut, donc bon.
Donc bon n'essayons pas d'aller vers le format le plus correct des deux ? (si la ligne a attiré mon attention c'est que l'espace avant les deux points, ça ne se fait pas, et si c'est passé avant, ça ne justifie pas de passer maintenant).
Mis à jour par Benjamin Dauvergne il y a presque 7 ans
- Lié à Development #16509: faire une fixture de HttpRequestMocking ajouté
Mis à jour par Benjamin Dauvergne il y a presque 7 ans
- Fichier 0001-handle-httplib.HTTPException-in-_http_request-16472.patch 0001-handle-httplib.HTTPException-in-_http_request-16472.patch ajouté
Voili, voilou, tests & correction à la description de l'exception pour coller au dernier patch dans le coin.
Ça dépend du #16509 pour éviter que HttpRequestMocking empêche de vraiment tester cet aspect (et puis c'est plus propre que les mocks soient installés quand on en a besoin et pas tout le temps).
Mis à jour par Thomas Noël il y a plus de 6 ans
- Statut changé de Nouveau à Rejeté
N'est plus d'actualité depuis #19437 qui passe à python-requests et gère requests.RequestException