Parcours de suppression de compte à l’aide du numéro de téléphone (#72615) #95
vdeniaud
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/72615-phone-based-account-deletion"
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?
0b3a6a95a0to133451f524133451f524to6d1e1fc4e66d1e1fc4e6to0daa1bd88c0daa1bd88ctof6781c06aff6781c06aftobaa5cbc752baa5cbc752to63aaf4dc1063aaf4dc10to98d30d835398d30d8353to43eb59ac5b43eb59ac5bto148499bab7148499bab7to7a87c06ebc7a87c06ebctob03c9cec50WIP: Parcours de suppression de compte à l’aide du numéro de téléphone (#72615)to WIP: Parcours de suppression de compte à l’aide du numéro de téléphone (#72615)b03c9cec50toffaf4b75d1ffaf4b75d1tof23ccc6b5ff23ccc6b5ftobc30e6267ebc30e6267eto764978701a764978701atod9397dbf372769d936c1to2e13264fc52e13264fc5to79ebd29a9679ebd29a96tode072747d4WIP: Parcours de suppression de compte à l’aide du numéro de téléphone (#72615)to Parcours de suppression de compte à l’aide du numéro de téléphone (#72615)Je pense qu’on pourrait déjà dans un premier temps passer ce parcours assez simple : la désactivation à l’aide du numéro de téléphone opérée uniquement lorsque celui-ci est connu et vérifié alors que le courriel ne l’est pas.
Pour info, en testant en local j'ai été bloqué par #79865 (le compte que j'ai créé par numéro de tel n'a pas d'email mais email_verified est vrai). J'ai corrigé dans un shell et le parcours fonctionne bien pour moi.
@ -2021,2 +2063,4 @@ctx = super().get_context_data(**kwargs)ctx['email'] = self.request.user.emailuser_ct = ContentType.objects.get_for_model(get_user_model())atvs = models.AttributeValue.objects.filter(Pourquoi pas la même requête que plus haut, avec with_owner ?
D'ailleurs cette requête est tellement courante désormais dans le code que ça donne bien envie de factoriser, avec genre un
user.get_phone_number(self.authenticator.phone_identifier_field).Oui tu as raison, c’est le moment d’en faire une méthode dédiée. Je vais revoir cela.
@ -2060,0 +2111,4 @@token = kwargs['deletion_token'].replace(' ', '')if token:try:with atomic():Ici j'imagine que le comportement souhaité c'est 1/ utiliser le token 2/ si il y a une erreur pendant l'utilisation, échouer mais réactiver le token.
Je pense que c'est inutile de chercher à faire ça, si le token est mauvais autant l'invalider (c'est comme ça partout ailleurs dans le code).
Concrètement ça veut juste dire enlever le bloc atomique, et revenir à une gestion normale des exceptions.
Oui tu as raison, pas besoin d’atomicité ici en fait. Je vais faire revoir cela.
Oui, j’avance de ce côté-ci et je gérerai ce souci de vérification abusive du courriel ensuite. Merci pour la relecture.
de072747d4to8c3e9c83208c3e9c8320to4a5490cef49259d22e9dtoc5e56d5970Voilà, tes deux commentaires ont été inclus. Nouvelle version bonne pour relecture :)
Top
@ -542,0 +543,4 @@@propertydef phone_identifier(self):if field := get_password_authenticator().phone_identifier_field:atvs = (Peut-être que ça marche de faire direct
return AttributeValue.objects.with_owner(...).first(), dans ce cas la requête serait j'imagine un peu plus performante, mais c'est du détailÇa me va de laisser comme cela, il n’est pas censé y avoir des tonnes de lignes renvoyées, si c’est le cas c’est qu’il y a un problème ailleurs.