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.

Sign up for GitHub

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

Jump to bottom

Ajout de l'authentification à deux facteurs #63

Closed
wants to merge 6 commits into from
Closed

Ajout de l'authentification à deux facteurs #63

wants to merge 6 commits into from

Conversation

Ash-Crow
Copy link
Collaborator

@Ash-Crow Ash-Crow commented Dec 7, 2023

🎯 Objectif

Ajout de la possibilité d’activer l'authentification à deux facteurs (2FA)

🔍 Implémentation

  • Ajout de l'extension wagtail-2fa
  • Ajout d’une variable d’environnement WAGTAIL_2FA_REQUIRED pour contrôler l’activation de cette fonctionnalité

⚠️ Informations supplémentaires

(optionnel) Documentation, commandes à lancer, variables d’environnement, etc.

🏕 Amélioration continue

  • Ajout d’une variable d’environnement SITE_NAME contrôlant le nom affiché à la fois sur le gestionnaire d’authentification, mais aussi sur le panneau d'admin du site

🖼️ Images

(optionnel) Une ou plusieurs captures d’écran, si pertinent

@Ash-Crow Ash-Crow added the amélioration Nouvelle fonctionalité ou demande label Dec 7, 2023
@Ash-Crow Ash-Crow linked an issue Dec 7, 2023 that may be closed by this pull request
Ajouter l'authentification 2FA #34
Open
@Ash-Crow
Copy link
Collaborator Author

Avis après test :

  • L'extension marche bien, même si je pense que demander aux utilisateurs de nommer eux-mêmes leur device complique l'utilisation sans apporter grand-chose (et puis on va se retrouver avec des tas de devices nommés "Téléphone")
  • Par contre : il n'y a pas de traduction française, et les mainteneurs semblent ne pas prendre en compte les contributions de la communauté (cf le repo https://github.com/LabD/wagtail-2fa où des PRs, notamment pour des traductions, sont ouvertes depuis des années.)

Je clos pour l'instant. On pourra réouvrir si on décide de faire un fork.

@Ash-Crow Ash-Crow closed this Dec 11, 2023
@Ash-Crow Ash-Crow mentioned this pull request Dec 11, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
amélioration Nouvelle fonctionalité ou demande
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Ajouter l'authentification 2FA
1 participant
@Ash-Crow