Développement #19437
closedMigrer les requêtes HTTP vers python-requests
Added by Frédéric Péters over 8 years ago. Updated about 7 years ago.
0%
Description
Ça uniformise et ça nous fait au passage gagner la prise en charge des proxy.
Files
Updated by Frédéric Péters over 8 years ago
- Related to Gestion de projet #19436: fonctionnement de Publik avec des proxies added
Updated by Frédéric Péters about 8 years ago
- Related to Développement #20089: saml : utiliser misc.http_post_request pour les appels SOAP added
Updated by Frédéric Péters about 8 years ago
- File 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch added
- Status changed from Nouveau to En cours
- Patch proposed changed from No to Yes
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)
Updated by Thomas Noël about 8 years ago
A priori on peut retirer le "import httplib" ; et ack.
Updated by Frédéric Péters about 8 years ago
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.
Updated by Frédéric Péters about 8 years ago
- File 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch added
- File 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch added
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.
Updated by Thomas Noël about 8 years ago
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
Updated by Frédéric Péters about 8 years ago
- File 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch added
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.
Updated by Frédéric Péters about 8 years ago
- File 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch 0002-misc-turn-urlopen-into-a-_http_request-wrapper-19437.patch added
- File 0001-misc-switch-http-support-to-requests-library-19437.patch 0001-misc-switch-http-support-to-requests-library-19437.patch added
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.
Updated by Frédéric Péters about 8 years ago
- Status changed from En cours to 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)
Updated by Frédéric Péters about 7 years ago
- Status changed from Résolu (à déployer) to Solution déployée