Support #79824
Du bon usage de models.Lock
Début:
19 juillet 2023
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
En tant que relecteur je n'ai pas trop su dire si l'utilisation du modèle Lock était correcte ici https://git.entrouvert.org/entrouvert/authentic/src/commit/6c27ec47dd5db2680759bcc25b044e4059788ace/src/authentic2/views.py#L429.
Ce ticket pour demander une relecture de Benjamin à posteriori.
Historique
Mis à jour par Benjamin Dauvergne il y a 8 mois
- Statut changé de Nouveau à Solution déployée
Ça me parait ok, l'important c'est que tous les endroits qui vont faire ce genre de requête :
non_unique = (
models.AttributeValue.objects.filter(
attribute=authn.phone_identifier_field,
content_type=user_ct,
object_id__isnull=False,
content=phone,
)
.exclude(object_id=user_pk)
.exists()
)
soient protégés par le même lock, si c'est le seul endroit où on fait ça c'est suffisant.
Mis à jour par Benjamin Dauvergne il y a 8 mois
- Assigné à changé de Benjamin Dauvergne à Valentin Deniaud