Development #19839
agent authentic : email unique par défaut
0%
Description
Dans #14747,
Discuté à l'instant (au bureau), on préfèrera que hobo déploie Authentic-publik avec le choix "email unique" coché par défaut.
Mais ça ne se fait pas pour l'agent de modifier le paramétrage de l'ou par défaut, elle existe dès le début et on n'aurait pas envie par la suite d'annuler un choix consciemment fait par l'admin.
On pourrait le faire pour les OU créées dans du contexte multi-collectivités mais d'une part ça ferait une différence de comportement bizarre, d'autre part les comptes des utilisateurs sont de toute façon créés sur l'OU par défaut.
Bref, je pense qu'on doit simplement poser le paramétrage côté settings.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de Nouveau à En cours
- Fichier 0001-authentic-agent-set-email-to-be-unique-by-default-19.patch 0001-authentic-agent-set-email-to-be-unique-by-default-19.patch ajouté
- Patch proposed changé de Non à Oui
Le relecteur intéressé pourra passer du temps côté authentic pour comprendre les interactions entre A2_EMAIL_IS_UNIQUE et A2_REGISTRATION_EMAIL_IS_UNIQUE et OrganizationalUnit.email_is_unique et les comportements présentés à l'usager.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Lié à Bug #14747: debian(multitenant) : set email is unique by default ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Sinon je propose ça:
diff --git a/hobo/agent/authentic2/management/commands/hobo_deploy.py b/hobo/agent/authentic2/management/commands/hobo_deploy.py
index f0e6030..3c6c5e4 100644
--- a/hobo/agent/authentic2/management/commands/hobo_deploy.py
+++ b/hobo/agent/authentic2/management/commands/hobo_deploy.py
@@ -38,6 +38,9 @@ class Command(hobo_deploy.Command):
# Activate default translation
activate(settings.LANGUAGE_CODE)
+ # Set email is unique on default OU
+ get_ou_model().filter(default=True).update(email_is_unique=True)
+
# create hobo users in authentic to bootstrap (don't update them,
# hobo is not a provisioning system)
for user_dict in hobo_environment.get('users'):
et le script suivant à lancer sur la prod/test/etc avec --all-tenant
from django_rbac.utils import get_ou_model() OU = get_ou_model() OU.filter(default=True).update(email_is_unique=True)
Mis à jour par Frédéric Péters il y a plus de 6 ans
Je notais « on n'aurait pas envie par la suite d'annuler un choix consciemment fait par l'admin. » (et j'entendais par "admin" la personne qui irait dans l'ui d'authentic cocher la case "emails uniques"); il me semble qu'avec ton patch on annulerait le choix conscient de l'admin dès un hobo_deploy.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Yep j'oubliais qu'il se ré-executait à l'identique à chaque fois, je n'ai pas trop d'idée là à part inventer un énième système de migration pour savoir dans quel état est l'instance, restons sur ton setting général, ça n'impacte que publik pour l'instant.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit a9840d1aa586c66a2080bd842da896ab857edf94
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Oct 31 20:09:57 2017 +0100
authentic agent: set email to be unique by default (#19839)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
authentic agent: set email to be unique by default (#19839)