Support #79824
Du bon usage de models.Lock
Start date:
19 July 2023
Due date:
% Done:
0%
Estimated time:
Patch proposed:
No
Planning:
No
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.
History
Updated by Benjamin Dauvergne over 1 year ago
- Status changed from Nouveau to 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.
Updated by Benjamin Dauvergne over 1 year ago
- Assignee changed from Benjamin Dauvergne to Valentin Deniaud