add double authentification for admin and directors #114

Open
opened 2026-05-21 17:57:21 +02:00 by unurled · 3 comments
Owner

Pull Request synced from GitHub: https://github.com/flambeaux-org/Sacadoc/pull/95
State: open | Base: mainHead: feature/81-add-double-auth
Author: @maurergilles | Created: 2026-02-23


Assez grosse PR, je vous laisse regarder ce que vous en pensez.

Pour l'issue #81

Capture d'écran 2026-02-23 164743 Capture d'écran 2026-02-23 164755 Capture d'écran 2026-02-23 165742
> **Pull Request synced from GitHub:** https://github.com/flambeaux-org/Sacadoc/pull/95 > **State:** open | **Base:** `main` ← **Head:** `feature/81-add-double-auth` > **Author:** @maurergilles | **Created:** 2026-02-23 --- Assez grosse PR, je vous laisse regarder ce que vous en pensez. Pour l'issue #81 <img width="1157" height="642" alt="Capture d&#39;écran 2026-02-23 164743" src="https://github.com/user-attachments/assets/0475425b-c42f-4071-8b9a-03354ee163bd" /> <img width="925" height="932" alt="Capture d&#39;écran 2026-02-23 164755" src="https://github.com/user-attachments/assets/049203f6-494a-4aeb-aabc-bbd52a8e8d9b" /> <img width="880" height="923" alt="Capture d&#39;écran 2026-02-23 165742" src="https://github.com/user-attachments/assets/1852b540-ccd7-42d5-9d05-dc38db1c56c6" />
Author
Owner

@maurergilles commented on GitHub:

je suis ouvert à toute demande de modifications fonctionnelles, ou affichage

<!-- gh-comment-id:3945830832 --> **@maurergilles** commented on GitHub: je suis ouvert à toute demande de modifications fonctionnelles, ou affichage
Author
Owner

@logut commented on GitHub:

Je me demande si utiliser https://github.com/jazzband/django-two-factor-auth ne serait pas plus simple. Ça nous permettrait aussi d'avoir l'envoi de mail supporté comme méthode de 2FA (ce que risque de choisir la plupart des personnes qui ne font pas de l'info)

<!-- gh-comment-id:3973554511 --> **@logut** commented on GitHub: Je me demande si utiliser https://github.com/jazzband/django-two-factor-auth ne serait pas plus simple. Ça nous permettrait aussi d'avoir l'envoi de mail supporté comme méthode de 2FA (ce que risque de choisir la plupart des personnes qui ne font pas de l'info)
Author
Owner

(inline review comment on feature/81-add-double-auth branch)
@mpeterschmitt commented on GitHub:

Pas sûr qu'on veuille que les directeurs et admin désactivent la 2FA.

@logut pourrais tu nous dire ce que tu en pense de la PR ?

<!-- gh-comment-id:2848615686 --> *(inline review comment on `feature/81-add-double-auth` branch)* **@mpeterschmitt** commented on GitHub: Pas sûr qu'on veuille que les directeurs et admin désactivent la 2FA. @logut pourrais tu nous dire ce que tu en pense de la PR ?
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mirror/sacadoc#114
No description provided.