Produits Entr'ouvert » Hobo

Le lien vers w.c.s. devrait passer par des slugs, je pensais que le ticket existait déjà mais non, je viens de le créer. (#7180)

url = self.wcs_url + 'roles?%s' % urllib.urlencode({'orig': self.orig})

C'est peut-être à changer mais aujourd'hui l'API exige un utilisateur.

role_extractor = role_extractors.WcsRoleExtractor

Même si je reste convaincu qu'abstraire prématurément est inutile, en gardant la structure actuelle, j'aurais quand même WcsRoleExtractor dans le fichier import-wcs-roles.

hobo/agent/authentic2/management/signature.py

Ça a quand même des chances de se retrouver utile de manière plus générale, ça pourrait aller dans hobo/ (dans utils.py?)

W.C.S.

w.c.s., en minuscules.

self.logger

Perdu sur une ligne ?

Ça mériterait aussi l'ajout de quelques mots sur les particularités de l'agent authentic dans le README (il y a déjà un bout sur wcs).

J'ai changé la commande pour générer tous les paramètres depuis le hobo.json autant que possible. Le seul truc un peu sioux c'est de trouver un email connu (ou même un NameID) dont on soit sûr qu'il existe sur l'instance de w.c.s. J'ai choisi de prendre le superuser le plus ancien. Ce code dépend maintenant du ticket #7192.

Frédéric Péters a écrit :

Le lien vers w.c.s. devrait passer par des slugs, je pensais que le ticket existait déjà mais non, je viens de le créer. (#7180)

J'ai modifié le code pour prendre ce futur slug comme identifiant.

url = self.wcs_url + 'roles?%s' % urllib.urlencode({'orig': self.orig})

C'est peut-être à changer mais aujourd'hui l'API exige un utilisateur.

Ajouté le paramètre email (voir remarque avant).

role_extractor = role_extractors.WcsRoleExtractor

Même si je reste convaincu qu'abstraire prématurément est inutile, en gardant la structure actuelle, j'aurais quand même WcsRoleExtractor dans le fichier import-wcs-roles.

Je voulais surtout séparer l'extraction de leur définition de leur création effective, j'ai tout réuni en une classe mais avec toujours plusieurs méthodes.

hobo/agent/authentic2/management/signature.py

Ça a quand même des chances de se retrouver utile de manière plus générale, ça pourrait aller dans hobo/ (dans utils.py?)

Déplacé dans hobo/.

W.C.S.

w.c.s., en minuscules.

Corrigé.

self.logger

Perdu sur une ligne ?

Enlevé.

---

Ça mériterait aussi l'ajout de quelques mots sur les particularités de l'agent authentic dans le README (il y a déjà un bout sur wcs).

Ajouté.

Frédéric Péters a écrit :

url = self.wcs_url + 'roles?%s'

Pour accompagner l'ajout des slugs il y a aussi la mise à disposition de l'API à l'adresse api/roles, plutôt que roles; l'ancien endpoint reste fonctionnel mais tant qu'à faire, autant utiliser le nouveau directement.

C'est pour tout de suite ?

Par ailleurs je me demande si c'est vraiment utile de nécessiter une requête authentifiée/signée pour ce endpoint, mais c'est une autre affaire.

Oui.

À part ça, en imaginant l'affaire testée, ce serait ok. (bon, en vrai il y a le "from .. import role_extractors" à retirer, puis import os et json à mettre, puis faire me = me⁰, puis ne pas passer sur les services qui ont un service-id != 'wcs', puis passer Accept: application/json, puis prendre en compte que la clé est créée sur une base ne contenant pas le port, et puis on arrive à une mise à jour à faire par rapport aux changements rbac dans authentic, parce qu'il n'y a plus de external_id, et ça je laisse faire).

Vu tout ça. J'ai fait un test d'intégration manuel de bout en bout, ça roule. Par contre je pense pousser l'extracteur directement dans authentic et seulement la commande dans hobo. C'est un peu chiant de tester avec hobo, je préfère faire mes tests en monotenant. Et si j'arrive à m'interposer dans les appels à requests j'aimerai bien avoir des tests.

url = self.wcs_url + 'roles?%s'

Pour accompagner l'ajout des slugs il y a aussi la mise à disposition de l'API à l'adresse api/roles, plutôt que roles; l'ancien endpoint reste fonctionnel mais tant qu'à faire, autant utiliser le nouveau directement.

C'est pour tout de suite ?

Ça a été mis en place avec la mise à disposition des slugs (#7180).

Vu tout ça. J'ai fait un test d'intégration manuel de bout en bout, ça roule. Par contre je pense pousser l'extracteur directement dans authentic et seulement la commande dans hobo. C'est un peu chiant de tester avec hobo, je préfère faire mes tests en monotenant. Et si j'arrive à m'interposer dans les appels à requests j'aimerai bien avoir des tests.

Je ne suis pas sûr de la séparation que ça va donner, tu mettrais signature.py et la classe WcsRoleImporter du côté d'Authentic, c'est ça ?

Si ça a été testé de bout en bout, c'est ok mais je suggérais url = self.wcs_url + 'roles?%s' % urllib.urlencode(..., que ça soit plutôt /api/roles qui soit appelé. Et print response.content à retirer.

Ok je pousse.