registration: perform proper identifier verification (#79865) #108
vdeniaud
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/79865-phone-authn-email-verification-bug"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
WIP: registration: perform proper identifier verification (#79865)to registration: perform proper identifier verification (#79865)J’ai laissé le
.set_email_verified(True, source='registration')ajouté à une époque où on voyait FranceConnect comme une source sûre pour le courriel, et où on souhaitait différencier la vérification qui en résultait de celle issue d’une création locale de compte. Ce n’est plus forcément le cas, ce sera peut-être à retirer dans un autre ticket, je ne sais pas encore, je laisse ça là en attendant.registration: perform proper identifier verification (#79865)to WIP: registration: perform proper identifier verification (#79865)Typo dans le test, terrible, je corrige.
dd455f4b53to84045e86e6WIP: registration: perform proper identifier verification (#79865)to registration: perform proper identifier verification (#79865)Je laisse deux commentaires pour expliciter ma compréhension des lignes pas directement en lien avec l'objet du ticket, à part ça nickel.
@ -1783,3 +1783,3 @@if hasattr(self, 'email'):attributes['email'] = self.emailif hasattr(self, 'phone'):Et ici il y avait une ligne inutile.
@ -1815,2 +1813,4 @@init_kwargs[key] = attributes[key]kwargs['instance'] = get_user_model()(**init_kwargs)# phone identifier is a separate attribute and is set post user-creationif hasattr(self, 'phone'):OK donc il y avait un deuxième bug, où le numéro de tel n'était lui pas marqué comme vérifié.
84045e86e6tofa45f190d068c74a7fd4toc317cf6a9b