Development #20106
newsletters: utiliser requests de utils
0%
Description
Qui se charge de faire la signature des requetes, etc.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans
- Fichier 0001-newsletters-use-utils.requests-to-sign-webservice-ca.patch 0001-newsletters-use-utils.requests-to-sign-webservice-ca.patch ajouté
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
Qu'est-ce qui amène la nécessité de désormais préciser status_code=200 (dans les mock.Mock et dans la classe MockResponse ? Et le passage de mock_get.call_args[0][0] à mock_get.call_args[0][1] ?
def get_newsletters(self, **kwargs):
[...]
response = requests.get(endpoint, remote_service='auto', cache_duration=0)
La liste des newsletters ne peut vraiment pas être mise en cache ?
endpoint = self.url + 'subscriptions/?' + urlencode(kwargs)
Utiliser params=payload plutôt que faire l'encodage ici.
- except requests.exceptions.HTTPError:
Il est décidé de ne plus offrir un log d'erreur différent dans ce cas, ok mais ça relève d'un autre commit.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans
- Fichier 0001-newsletters-use-utils.requests-to-sign-webservice-ca.patch 0001-newsletters-use-utils.requests-to-sign-webservice-ca.patch ajouté
Frédéric Péters a écrit :
Qu'est-ce qui amène la nécessité de désormais préciser status_code=200 (dans les mock.Mock et dans la classe MockResponse ? Et le passage de
mock_get.call_args[0][0]àmock_get.call_args[0][1]?
C'est le passage à utils.requests.
J'ai rajouté un test à part pour vérifier que les appels sont bien signés.
La liste des newsletters ne peut vraiment pas être mise en cache ?
Si, une minute me parait bien.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de Nouveau à En cours
C'est le passage à utils.requests.
Mais là c'est la description du ticket :/ (j'imaginais une précision, genre "on passe par une classe donc le premier argument c'est self et les autres décalés", (et aucune idée pour le 200)). Laissons tomber.
Ack.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans
Le status_code dans la réponse mockée est nécessaire lors de l'usage de la classe Requests. C'est fait dans d'autres tests.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 26d87427000c803a1ae7c10f36cc091832c0d7b2 (origin/master)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Wed Nov 15 19:17:28 2017 +0100
newsletters: use utils.requests to sign webservice calls (#20106)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans
- Statut changé de Résolu (à déployer) à Fermé
newsletters: use utils.requests to sign webservice calls (#20106)