Development #33392
on n'a pas perdu le support de l'authentification HTTP Basic dans http_request
Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Version cible:
-
Début:
23 mai 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Depuis
commit 218bc1063bf7d2de670592ccbaaaa436fc7dc15f Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Nov 14 23:18:55 2017 +0400 misc: switch http support to requests library (#19437)
la variable auth n'est branchée sur rien :
if '@' in hostname: authenticator, hostname = hostname.split('@') if ':' in authenticator: username, password = authenticator.split(':', 1) else: username = authenticator password = '' auth = (username, password) timeout = timeout or settings.REQUESTS_TIMEOUT try: response = requests.request(method, url, headers=headers, data=body, timeout=timeout, cert=cert_file, proxies=settings.REQUESTS_PROXIES) except requests.Timeout:
Mais ça marche quand même car requests le gère tout seul :
Python 2.7.15+ (default, Feb 3 2019, 13:13:16) [GCC 8.2.0] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import requests >>> requests.get('http://httpbin.org/basic-auth/login/password') <Response [401]> >>> requests.get('http://login:password@httpbin.org/basic-auth/login/password') <Response [200]> >>>
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
Et donc, il y a juste du code inutile, c'est ça ?
Mis à jour par Frédéric Péters il y a presque 5 ans
- Sujet changé de on a perdu le support de l'authentification HTTP Basic dans http_request à on n'a pas perdu le support de l'authentification HTTP Basic dans http_request
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-trivial-remove-obsolete-HTTP-auth-code-33392.patch 0001-trivial-remove-obsolete-HTTP-auth-code-33392.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Oui j'ai vu ça en passant, je cherchais l'origine du bug coté Nanterre mis sur une mauvaise piste par Thibault, je l'ai mis en priorité bas volontairement.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Et désolé pour le titre j'étais parti pour dire que ça ne marchait plus, puis je me suis dit que ça se serait vu tout en écrivant la description et puis pouf pouf.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6dd9433f64d63ed39d47e57a7b426f46123eb006 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu May 23 21:14:52 2019 +0200 trivial: remove obsolete HTTP auth code (#33392)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
trivial: remove obsolete HTTP auth code (#33392)