Produits Entr'ouvert » Publik

Échange avec fred:

Fred Peters 10:56:34
yo benj, tu penses quoi de https://dev.entrouvert.org/issues/7769 ?
bdauvergne@im.libre-entreprise.com 10:58:43
toujours la même remarque sur le jsonp mais oui, par contre c'est embêtant si je renvois la liste de tous les services avec éventuellement une liste de rôles vides ?
Fred Peters 11:00:48
je ne pense pas, non; par contre j'étais parti sur l'idée qu'on matcherait les services en utilisant leur saml metadata url mais en fait ça peut juste être leur slug.
bdauvergne@im.libre-entreprise.com 11:01:27
de toute façon j'essaierai de renvoyer un max d'information en mettant un to_json() sur l'objet Service de base puis aussi sur LibertyProvider { 'name': , 'slug': , 'metadata_url':, 'roles': [...], etc..}
Fred Peters 11:02:32
ça me semble ok.
bdauvergne@im.libre-entreprise.com 11:03:39
quand je vais m'attaquer à la gestion des accès coté idp pour les services je vais ajouter une checkbox "Ouvert à tous" à l'objet Service qui provoquera la création d'un rôle "Accès", si on a pas ce rôle le service ne sera pas renvoyé dans le jsonp, comme ça on couvrira tous les cas
Fred Peters 11:04:45
ça marche

Une première implémentation, qui fournit du JSONP ou du JSON avec check Referer ou CORS.

Les deux premiers patchs sont des ajouts au décorateur de cache pour mettre en cache la validation des origines.
Le dernier contient l'implémentation d'un décorateur json équivalent de to_json de passerelle, d'un système d'autorisation des origines basés sur les sessions SAML (l'origine de l'entity ID d'un provider avec une session en cours doit matcher l'origine ou le referer de la requête), et une vue user_info/ basé sur des méthodes to_json() ajouté aux classes User, Role et Service.

Fonctionnellement pour moi ça marche (patch au publik.js attaché, testé et validé avec les patchs authentic appliqués en local).

Patch côté combo (qui par rapport au précédent retire un console.log()).

commit b98bedebecae37ac04ce679aa9821d1c03f5f96b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jul 8 09:13:27 2015 +0200

    publik: look deeper for relevant wcs instance (#7769)

    If there are several instances of wcs deployed (which is typical of a
    multi-collectivity deployment) we ask authentic for user details so we
    know where the user has been given roles, we can then get the relevant
    wcs.

A regarder les commits dans authentic, l'url pour récuperer les infos utilisateur est /api/user/ et non /user_info/

Merci Serghei pour l'attention; Benjamin, étant donné que les commits dans authentic se font sans temps de relecture, sur ce genre de différence, touchant du code déjà écrit, ça serait cool d'avoir une notification.

commit b10e33501646569b67a12ca281d320ce852ba26a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Jul 16 09:59:22 2015 +0200

    publik: adapt to change in authentic user api endpoint url (#7769)