Produits Entr'ouvert » Authentic 2

On est fixé sur #75138 et #75139, desquels dépend celui-ci. Je commence à regarder.

Paul Marillonnet (pmarillonnet) a ouvert une pull request sur Gitea concernant cette demande :

• URL : https://git.entrouvert.org/entrouvert/authentic/pulls/50
• Titre : WIP: place service in context processor (#75142)
• Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/50/files

Les services sont déjà posés en session au SSO et disponibles dans le contexte des templates :

src/authentic2/idp/saml/saml2_endpoints.py:                set_service(request, provider_loaded)
src/authentic2/idp/saml/saml2_endpoints.py:    set_service(request, provider)
src/authentic2/idp/saml/saml2_endpoints.py:    set_service(request, provider)
src/authentic2/utils/service.py:def set_service(request, service):
src/authentic2/utils/service.py:            set_service(request, Model.objects.get(pk=pk))
src/authentic2_idp_cas/views.py:        set_service(request, model)
src/authentic2_idp_cas/views.py:                set_service(request, model)
src/authentic2_idp_oidc/views.py:        set_service(request, client)
src/authentic2_idp_oidc/views.py:                set_service(request, provider)

# authentic2/context_processors.py
def a2_processor(request):
    global __AUTHENTIC2_DISTRIBUTION  # pylint: disable=global-statement
    variables = {}
    variables.update(app_settings.TEMPLATE_VARS)
    variables['federations'] = UserFederations(request)
    if __AUTHENTIC2_DISTRIBUTION is None:
        if settings.DEBUG:
            __AUTHENTIC2_DISTRIBUTION = repr(get_distribution('authentic2'))
        else:
            __AUTHENTIC2_DISTRIBUTION = str(get_distribution('authentic2'))
    variables['AUTHENTIC2_VERSION'] = __AUTHENTIC2_DISTRIBUTION
    if hasattr(request, 'session'):
        variables['LAST_LOGIN'] = request.session.get(constants.LAST_LOGIN_SESSION_KEY)
        variables['USER_SWITCHED'] = constants.SWITCH_USER_SESSION_KEY in request.session
        service = get_service(request)
        if service:
            variables['service'] = service
            variables['service_colour'] = service.colour
            # name and logo should be show for OIDC service only
            variables['show_service_infos'] = type(service).__name__ != 'LibertyProvider'

            if service.logo:
                variables['service_logo_url'] = service.logo.url
            if service.ou:
                variables['service_ou_colour'] = service.ou.colour
                if service.ou.logo:
                    variables['service_ou_logo_url'] = service.ou.logo.url
    return variables

On vise à pallier à quoi ici ?

Benjamin Dauvergne a écrit :

Les services sont déjà posés en session au SSO et disponibles dans le contexte des templates :
[...]
et
[...]

On vise à pallier à quoi ici ?

Ok, il y a sans doute un bout de code superflu dans le patch, je regarde ce que je peux retirer.
Ça n’invalide pas la nécessité de placer dans le contexte de gabarit de la page “Mon compte” la configuration d’affichage générique, qui est l’objet principal de ce ticket.

Autre questionnement (pour éviter de s’y reprendre à deux fois comme pour #75139 puis #77296), dans quels cas précisément la personnalisation de cette page “Mon compte” doit avoir lieu. À relire les spécifications écrites à ce sujet, il me semble qu’il n’y en a que deux :
· si une configuration d’apparence générique est définie, et que le service est un service interne SAML, alors on applique cette apparence ci ;
· si le service est un service tiers OIDC, alors on applique de préférence la configuration d’apparence locale au service OIDC, sinon celle de son OU d’appartenance.

Automatic expiration