Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connexion automatique par lien #29

Merged
merged 6 commits into from
Oct 28, 2024
Merged

Connexion automatique par lien #29

merged 6 commits into from
Oct 28, 2024

Conversation

ikarius
Copy link
Contributor

@ikarius ikarius commented Oct 26, 2024

Connexion automatique par "liens magiques"

voir : https://trello.com/c/RwEf5XrL/206-front-lien-magique-proconnect

Permet à un utilisateur avec un ancien compte Inclusion Connect de se connecter directement à DORA via un lien magique.

Le lien est :

  • à usage unique,
  • valable 5 minutes.

Utilisation

Sur la page de connexion ProConnect, il y a un lien "Des difficultés pour vous connecter ?".

L'utilisateur est redirigé vers une modale permet de saisir une adresse e-mail pour la réception d'un lien de connexion.

Si l'utilisateur est bien titulaire d'un ancien compte Inclusion Connect lié à DORA, il reçoit un e-mail contenant le lien pour une connexion directe.

Dépendances

Cette fonctionnalité est basée sur la librairie django-sesame : https://github.com/aaugustin/django-sesame

Une nouvelle application Django auth_links regroupe toute les fonctionnalités pour la gestion des liens.

Pour la gestion des connexions par liens ("liens magiques").
@ikarius ikarius force-pushed the sesame-magic-links branch from 689f2db to 81089a3 Compare October 26, 2024 10:25
@ikarius ikarius requested review from ggounot and removed request for ggounot October 26, 2024 10:28
@ikarius
Copy link
Contributor Author

ikarius commented Oct 26, 2024

Note : pas encore en revue "officielle" pour vérifier certains détails (comme le contenu de l'e-mail)

@ikarius ikarius self-assigned this Oct 27, 2024
@ikarius ikarius force-pushed the sesame-magic-links branch 4 times, most recently from 15d52bc to af5de12 Compare October 27, 2024 16:33
@ikarius ikarius marked this pull request as ready for review October 27, 2024 16:36
Pour la gestion des liens "magiques" de connexion.
Nouvelle possiblité de connexion à DORA, pour l'instant uniquement pour
les comptes anciennement connectés avec Inclusion Connect.
Modification du flow OIDC pour permettre une déconnexion sans token OIDC
de session (absent lors d'une connexion par lien)
… lien

L'ancien lien d'aide est temporaire modifié (et est devenu modifiable
dans le composant)
@ikarius ikarius force-pushed the sesame-magic-links branch from af5de12 to f8ee8a9 Compare October 27, 2024 16:42
Haut niveau seulement, et pas de tests qui pourraient être de la
responsabilité de `django-sesame`
@ikarius ikarius force-pushed the sesame-magic-links branch from 48346e0 to 40a6b36 Compare October 27, 2024 17:37
@ikarius ikarius requested review from ggounot and jbuget October 27, 2024 17:41
@ikarius ikarius added the feature New feature or request label Oct 28, 2024
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bravo pour le travail et la qualité du code (et les tests)

Copy link
Contributor

@ggounot ggounot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top ! Code et tests impeccables 👏

Idées d'amélioration :

  • message d'erreur plus précis que Erreur de connexion ;
  • limiter dans le temps le nombre de demandes de lien magique successives, mais ce n'est pas vraiment problématique.

@ikarius
Copy link
Contributor Author

ikarius commented Oct 28, 2024

@ggounot Merci pour tes remarques, je les prends en compte pour une PR prochaine (j'ai comme l'impression que la connexion va rester dans le coin).

@ikarius ikarius merged commit 6aa304d into main Oct 28, 2024
7 checks passed
@ikarius ikarius deleted the sesame-magic-links branch October 28, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants