Produits Entr'ouvert » Authentic 2

En tout cas oui chouette idée :)

Je précise donc dans le titre de celui-ci que je parlais côté SP, et j'ai créé un autre pour le côté IdP (#37884).

Allez, une première proposition. Qui ne tient pas compte de la réquisition (OIDCClaimMapping.required) et de la vérification (OIDCClaimMapping.verified) lorsque la revendication est exprimée en langage de gabarit, parce que ça me paraît arbitraire (l'attribut devient vérifié si toutes les revendications dans l'expression de gabarit le sont ? ou une seule ? même question pour l'état de réquisition).
Si le cas d'usage à l'origine du ticket impose le support de ces deux concepts, alors je reverrai ma copie (sans doute par ajout de code d'inspection de la chaîne de gabarit, d'extraction des variables et de comparaison avec le contenu du jeton ID et du payload UserInfo) une fois qu'on se sera mis d'accord sur le côté arbitraire dont je parle ici.

Benjamin Dauvergne a écrit :

Le patch 4 si provider=None, parsed n'est pas défini; provider n'est pas optionnel je pense.

Ouch oui en effet. Corrigé dans la branche et ici.

Benjamin Dauvergne a écrit :

C'est pareil

C'est rouge mais cette fois-ci j'y suis pour rien :)
On dirait un souci de dépendance sur jenkins, je reproduis pas en local cette erreur de syntaxe (code python3 dans un venv python2) :

Traceback (most recent call last):
    […]
    app_config = AppConfig.create(entry)
  File "/tmp/jenkins-authentic-wip-wip%2F37871-auth-oidc-template-mapping-19/tox-jenkins/authentic/py27-coverage-dj111-authentic-pg-oldldap/lib/python2.7/site-packages/django/apps/config.py", line 94, in create
    module = import_module(entry)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/var/lib/jenkins/workspace/37871-auth-oidc-template-mapping/src/authentic2/manager/__init__.py", line 18, in <module>
    import django_select2.conf  # noqa: F401
  File "/tmp/jenkins-authentic-wip-wip%2F37871-auth-oidc-template-mapping-19/tox-jenkins/authentic/py27-coverage-dj111-authentic-pg-oldldap/lib/python2.7/site-packages/django_select2/conf.py", line 5, in <module>
    from appconf import AppConf
  File "/tmp/jenkins-authentic-wip-wip%2F37871-auth-oidc-template-mapping-19/tox-jenkins/authentic/py27-coverage-dj111-authentic-pg-oldldap/lib/python2.7/site-packages/appconf/__init__.py", line 1, in <module>
    from .base import AppConf  # noqa
  File "/tmp/jenkins-authentic-wip-wip%2F37871-auth-oidc-template-mapping-19/tox-jenkins/authentic/py27-coverage-dj111-authentic-pg-oldldap/lib/python2.7/site-packages/appconf/base.py", line 107
    class AppConf(metaclass=AppConfMetaClass):
                           ^
SyntaxError: invalid syntax

pourquoi if provider ?

Parce que je ne me relis pas assez :/
Corrigé ici.

Paul Marillonnet a écrit :

Parce que je ne me relis pas assez :/
Corrigé ici.

Ah oui, si quand même, c'est parce que dans parse_id_token si provider n'a pas d'attribut jwkset ça va lever une AttributeError. Mais c'est de sa faute :)
Je vais corriger.

Quelque chose comme ça me paraît plus logique, et notamment plus raccord avec la docstring de parse_id_token.

Ça ne sert à rien de lever des exceptions pour des trucs qui ne doivent jamais arriver.

Ok j'ai viré ce bout de code.

commit 2b46edf4c578c124a29a18cc95ec9ca7a6f1b869
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Mar 13 16:21:03 2020 +0100

    auth_oidc: render templated claim values during authn (#37871)

commit 556f3e169e28da0f30b890c056593f60a7f713a8
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Mar 30 11:46:14 2020 +0200

    auth_oidc: add id token 'as_dict' method (#37871)

commit 7991c4869ed74a5754bd321f58c54d85875af906
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Mar 27 17:07:24 2020 +0100

    auth_oidc: select from existing attributes in admin provider page (#37871)

commit c4636a41cef5987204d5c2f9e7ef765537ad12aa
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Mar 13 15:01:12 2020 +0100

    auth_oidc: use custom widget in claim mapping admin form (#37871)

commit dccf9a571f1fdfe519ed7e7bb08bcd57151852c3
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Tue Mar 17 17:32:46 2020 +0100

    auth_oidc: extend mapping claim max length (#37871)