Produits Entr'ouvert » Authentic 2

Disclaimer : fonctionnellement je nage même si j'ai parcouru la spec.

Typo logout_list devrait s'appeler redirect_logout_list ?

Pourquoi le code de add_oidc_session gère potentiellement plusieurs oidc_session pour une même session/requête ?

value.get('frontchannel_timeout') or 10000 : plutôt dans de la conf le 1000 ?

Je ne vois aucun endroit où les nouveaux champs de ton modèle sont utilisés lors d'une instanciation, c'est planqué quelque part avec de l'introspection de modèle ?

Emmanuel Cazenave a écrit :

Disclaimer : fonctionnellement je nage même si j'ai parcouru la spec.

Typo logout_list devrait s'appeler redirect_logout_list ?

Non il y a deux méthodes, ça ne servait à rien d'implémenter redirect_logout_list ici.

Pourquoi le code de add_oidc_session gère potentiellement plusieurs oidc_session pour une même session/requête ?

Pour chaque service en OIDC sur lequel on s'est connecté il faut générer une déconnexion.

value.get('frontchannel_timeout') or 10000 : plutôt dans de la conf le 1000 ?

Ok.

Je ne vois aucun endroit où les nouveaux champs de ton modèle sont utilisés lors d'une instanciation, c'est planqué quelque part avec de l'introspection de modèle ?

Si tu parles des tests oui c'est la liste OIDC_CLIENT_PARAMS qui paramètre les tests.

ok pour moi.