Development #10838
cellule newsletters: envoyer l'uuid dans les requetes vers l'API des newsletters
0%
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Fichier 0001-newsletters-add-uuid-param-for-webservice-calls-1083.patch 0001-newsletters-add-uuid-param-for-webservice-calls-1083.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 8 ans
Les modifs à models.py et test_newsletters_cell.py concernent autre chose ?
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
C'est une adaptation aux réponses de corbo qui ne sont pas formattés avec to_json
, comme celles du connecteur meyzieu_newsletters dans passerelle.
Mis à jour par Frédéric Péters il y a environ 8 ans
Ok, donc c'est plutôt pour un autre commit, qui ne serait pas "add uuid param for webservice calls".
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Fichier 0001-newsletters-add-uuid-param-for-webservice-calls-1083.patch 0001-newsletters-add-uuid-param-for-webservice-calls-1083.patch ajouté
- Fichier 0002-newsletters-check-response-by-status-code-instead-of.patch 0002-newsletters-check-response-by-status-code-instead-of.patch ajouté
Ok
Mis à jour par Frédéric Péters il y a environ 8 ans
Les connecteurs existants, ils fonctionnent par code de retour HTTP ? (je regarde celui Meyzieu et il me semble que non)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
C'est to_json
qui est en charge du code de retour.
Si tout se passe bien, le connecteur Meyzieu renvoie toujours 200 avec un json.
Mis à jour par Frédéric Péters il y a environ 8 ans
Si tout se passe bien [...]
C'est bien mon propos d'également faire attention à ne pas casser le traitement des erreurs. Je partais donc de l'idée que comme on changeait la gestion des erreurs ici, elle devait également être adaptée de l'autre côté.
Cela étant; en cas d'erreur, le set_subscriptions des newsletter meyzieu fait un return False et le résultat ça va être une réponse {"data": false, "err": 0}
et un code 200. Si je lis correctement. Et donc me soucier de la compatibilité de la gestion d'erreurs alors qu'il n'y avait rien, c'est futile.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
Yep, il faudrait gérer proprement l'erreur du côté du connecteur: #10853
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
Voilà, le code de retour est géré du côté du connecteur.
Chez moi en local ça fonctionne.
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
Ok ta remarque concernait le coté passerelle.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
Ce ticket était en attente de la gestion correcte des réponses de la part du connecteur meyzieu_newsletters dans passerelle. Comme c'est fait et poussé, je relance sur celui-ci
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 5820af546d392b80b665b79b527107ea08977445 Author: Serghei Mihai <smihai@entrouvert.com> Date: Fri May 6 11:48:48 2016 +0200 newsletters: check response by status code instead of json attribute commit 6ca4e35ce6cfb9ee80e878e3dec6ac13e9e444be Author: Serghei Mihai <smihai@entrouvert.com> Date: Fri May 6 11:08:09 2016 +0200 newsletters: add uuid param for webservice calls (#10838)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de Résolu (à déployer) à Fermé
newsletters: add uuid param for webservice calls (#10838)