Produits Entr'ouvert » Authentic 2

Je voudrais voir #72519 corrigé avant de décider de ce qu'on fait ici.

Ok mais enrober le message brut pas contrôlé qu'on peut recevoir reste une bonne idée.

Benjamin Dauvergne a écrit :

Je voudrais voir #72519 corrigé avant de décider de ce qu'on fait ici.

Apparemment c’est fait, l’occasion de remettre ce ticket ci sous les projecteurs :)

• si prompt=none on log rien, on ne s'attendait pas à ce que le sso fonctionne, ça foire, tant pis,
• si on a pas prompt=none alors bien sur on peut logger via logging mais surtout il faudrait enregistrer dans le journal une user.login.failure :
  

       request.journal.record('user.login.failure', ...) # ici on ne sait pas trop quoi mettre comme contexte
  

  
  on ne peut pas vraiment y indiquer d'attributs de contexte vu qu'on a rien à part une erreur mais donc il faudrait modifier la définition du type d'èvènement login.failure pour que username ne soit plus requis mais qu'on y enregistre les références au mode d'authentification ainsi que le message d'erreur précis remonté. Ça permettra une éventuelle analyse à posteriori par un admin.

# src/authentic2/journal_event_types.py
class UserLoginFailure(EventTypeWithService):
    name = 'user.login.failure'
    label = _('login failure')

    @classmethod
    def record(cls, *, service, username, user, reason=None):
        super().record(user=user, service=service, data={'username': username, 'reason': reason})

    @classmethod
    def get_message(cls, event, context):
        username = event.get_data('username')
        reason = event.get_data('reason')
        msg = _('login failure with username "{username}"').format(username=username)
        if reason:
            msg.append(_(' (reason: {reason})').format(reason=reason))
        return msg

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

• URL : https://git.entrouvert.org/entrouvert/authentic/pulls/39
• Titre : WIP: auth_oidc: provide a less scary displayed error message (#73669)
• Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/39/files

Automatic expiration