Development #36888
cart@ds, synchroniser rôle authentic et "abonnés" cart@ds
0%
Description
Pour permettre de gérer des accès aux détails de la demande dans w.c.s.
J'ai pas mal hésité à considérer ça spécifique et à le mettre dans les bouts spécial GL, et puis, non.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 4 ans
Au moment de l'appel à relationships, peut-être veux-tu vraiment synchroniser, donc retirer d'éventuels anciens uuid, dans ce cas il faudrait appeler PATCH plutôt que POST ?
Mis à jour par Thomas Noël il y a plus de 4 ans
Aussi, je n'ai pas bien compris le commentaire "this can fail" dans sync_subscribers_role ... qu'est-ce qui pourrait ne pas fonctionner ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch ajouté
En fait tout le temps je veux juste totalement synchroniser, j'ai juste pas vraiment lu ce qui était fait par l'API...
Mis à jour par Frédéric Péters il y a plus de 4 ans
Aussi, je n'ai pas bien compris le commentaire "this can fail" dans sync_subscribers_role ... qu'est-ce qui pourrait ne pas fonctionner ?
Authentic, de manière générale.
Mis à jour par Thomas Noël il y a plus de 4 ans
Tu as modifié le mauvais POST (c'est le second qu'il faut remplacer par PATCH).
"this can fail": en cas de pépin, on pourrait avoir des exceptions lors des role_uuid = response.json()['uuid'] par exemple. Est-ce que tu envisagerais alors de faire un gros try/except sur sync_subscribers_role pour éviter de faire crasher les endpoints qui l'appelle ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch 0001-cartads-cs-sync-subscribers-with-authentic-role-3688.patch ajouté
Tu as modifié le mauvais POST (c'est le second qu'il faut remplacer par PATCH).
Misère…
"this can fail": en cas de pépin, on pourrait avoir des exceptions lors des role_uuid = response.json()['uuid'] par exemple. Est-ce que tu envisagerais alors de faire un gros try/except sur sync_subscribers_role pour éviter de faire crasher les endpoints qui l'appelle ?
Je ne voulais pas faire un truc trop vague, j'ai adapté pour attraper ce qui peut arriver sur un response.json()['uuid'] (ValueError ça correspond au cas JSONDecodeError).
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Le dernier "response = " ne sert à rien, mais zou.
Comme j'aime bien parler, je parle : l'attribution des rôles ne sera pas instantanée dans wcs ou combo (le temps de propagation depuis A2), il faudra éventuellement s'en rappeler...
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 99d0eb02a4438c53859b3002d373dc90c39a051f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Oct 9 20:54:14 2019 +0200
cartads cs: sync subscribers with authentic role (#36888)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
cartads cs: sync subscribers with authentic role (#36888)