Development #19437
Migrer les requêtes HTTP vers python-requests
0%
Description
Ça uniformise et ça nous fait au passage gagner la prise en charge des proxy.
Fichiers
Demandes liées
Révisions associées
misc: turn urlopen into a _http_request wrapper (#19437)
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Lié à Project management #19436: fonctionnement de Publik avec des proxies ajouté
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Lié à Development #20089: saml : utiliser misc.http_post_request pour les appels SOAP ajouté
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Fait parce que ça m'aurait été utile pour avancer dans une installation sans avoir de certificats valides. (plus facile de simplement hacker python-requests pour désactiver la vérification des certificats)
Mis à jour par Thomas Noël il y a plus de 6 ans
A priori on peut retirer le "import httplib" ; et ack.
Mis à jour par Frédéric Péters il y a plus de 6 ans
J'avais juste cherché httplib mais il y a un tas de requêtes qui sont faites via urlopen, que je dois aussi convertir; nouveau patch à venir.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch ajouté
- Fichier 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch ajouté
Nouvelle version avec les import retirés; et patch qui ajoute un misc.urlopen() qui fournit une signature similaire à urllib2.urlopen, et utilisation de cette version partout.
Mis à jour par Thomas Noël il y a plus de 6 ans
0001 : ok
0002 :
Dans misc.urlopen :
'POST' if data else 'GET'va faire un GET si data est {} ou []. Remplacer par
'POST' if data is not None else 'GET'ou plutôt
'GET' if data is None else 'POST'@Dans wcs/api.py il faut conserver le read() : misc.urlopen(url) → misc.urlopen(url).read()
Sinon ça semble ok
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch ajouté
La réponse mockée dans les tests contenait une méthode getheader() ce qui fait que je passais à côté de sa disparition; correction ici.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch ajouté
- Fichier 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch ajouté
Il manque le deuxième patch.
Parce qu'il n'existait tout simplement pas; la mise à jour de 0001 ayant été effectuée indépendamment de la relecture par Thomas.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 0773410d026d8893fa09c9f8f1fe5b59ffc0e4a5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Nov 15 14:54:38 2017 +0400
misc: turn urlopen into a _http_request wrapper (#19437)
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)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: switch http support to requests library (#19437)