feat: add auto_validate_new_user boolean field#143
feat: add auto_validate_new_user boolean field#143jacquesfize merged 3 commits intoPnX-SI:developfrom
Conversation
edelclaux
force-pushed
the
feat/add-manual-validation-behavior
branch
from
81cff23 to
2eae584
Compare
|
Suite à réunion du jour |
| load_default="preferred_username" | ||
| ) # Claim d’identification du token OpenID/OIDC | ||
| RECONCILIATE_ATTR = fields.String(load_default="email") | ||
| AUTO_VALIDATE_NEW_USER = fields.Boolean(load_default=False) |
There was a problem hiding this comment.
Le fonctionnement par défaut est donc modifié (jusque là, le user se créait automatiquement). Toutefois, c'est le fonctionnement par défaut dans la connexion classique AUTO_ACCOUNT_CREATION=false, donc je pense que c'est bien de faire comme tu as fait et qu'il faudra juste le notifier dans le changelog.
There was a problem hiding this comment.
a la base, c'était une erreur ! J'ai mis faux pour le dev, j'ai juste oublié de remettre à vrai pour la PR.
Je laisse comme ça du coup ?
There was a problem hiding this comment.
Oui, ça me semble bien de le laisser comme ça. Il faut juste mettre à jour la description de la PR où tu dis :
Ajout de l'entrée de config "auto_validate_new_user", vraie par défaut.
En notifiant bien que le comportement à changer pour mettre en cohérence avec celui de l'auth de base.
| error_description = exc.description or "Unauthorized" | ||
| login_url = f"{current_app.config['URL_APPLICATION']}/#/login" | ||
| query_params = { | ||
| "login_error": error_description, |
There was a problem hiding this comment.
Pour l'instant pas de prise en compte dans le frontend de GeoNature de ce querry param, donc, quand on essaye de créer un compte avec AUTO_VALIDATE_NEW_USER à false, ça nous renvoie sur la page sans message de succès ou d'erreur.
Par ailleurs, pour ce cas spécifique, est-ce que c'est vraiment une erreur que l'on veut afficher ? Il semblerait plus logique de mettre un succès ou une info reprennant MyAccount.Message.AdminAccountEmailConfirmation ("Votre demande de création de compte a bien été prise en compte, elle va être évaluée par un administrateur")
There was a problem hiding this comment.
ah tiens, j'avais oublié de faire la PR. C'est chose faite :)
PnX-SI/GeoNature#3976
J'ai opté pour l'erreur pour se brancher au plus simplement sur la page existante.
Je suis ouvert à d'autres options, mais j'ai peur que ce soit plus/trop impactant
There was a problem hiding this comment.
Je regarde ça avec ton autre PR, merci :)
There was a problem hiding this comment.
Bon en effet, ça demande beaucoup de modification, j'ai essayé de le faire et je me suis un peu lancé dans un tunnel ... J'ai fais deux PR en rapport naturalsolutions/geonature#10, et naturalsolutions#1 à voir ce qu'on en fait
There was a problem hiding this comment.
Le travail de @christophe-ramet sera merge après le merge de cette branche
edelclaux
force-pushed
the
feat/add-manual-validation-behavior
branch
from
aa387c5 to
a0b2260
Compare
3 participants
Closes #142
Ajout de l'entrée de config "auto_validate_new_user", vraie par défaut.
L'appel à insert_or_update_role dans authorize a été étoffé.
Si
auto_validate_new_userest à vrai --> comportement actuel.Sinon:
raise Unauthorized("Demande de creation de compte en attente de validation.")raise Unauthorized("Demande de creation de compte créée et en attente de validation.")