authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786) #287
tnoel
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/88786-phone-authn-sms-code-duration-through-authenticator"
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?
bfbe4449c9to5ce081ecbc115d7f2a8ato16ad3236e116ad3236e1to5c9ec86f0396a3e2eba3to264073a783WIP: authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)to authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)@ -409,1 +410,4 @@)sms_code_duration = models.PositiveSmallIntegerField(_('SMS codes lifetime (in seconds)'),default='180',En posant un default ici, ça fait qu'il n'y aura pas de valeur par défaut qu'on pourra changer ensuite chez tous nos clients. Il serait préférable d'avoir un settings.SMS_CODE_DURATION qui sera la durée à prendre en compte si sms_code_duration est à None (quand aucune valeur n'aura été posée). Genre
self.sms_code_duration or getattr(settings, 'SMS_CODE_DURATION', None) or 180Ok très bien, je vais faire ainsi.
C’est corrigé et visible dans la branche.
authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)to WIP: authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)264073a783to705791f1f0ffd6e5db6ato0687a4c06f0687a4c06fto4f8099d2964f8099d296to6f01cc73cc6f01cc73cctoa70de53b22WIP: authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)to authn tél : rendre paramétrable dans le BO la durée de vie des codes SMS reçus (#88786)@ -428,1 +429,4 @@)sms_code_duration = models.PositiveSmallIntegerField(_('SMS codes lifetime (in seconds)'),help_text=_('Time (in seconds, between 60 and 3600) after which SMS codes expire.'),Ajoutons "la valeur par défaut est " settings.SMS_DURATION_MACHIN
C’est fait, avec un help_text qui est maintenant un callable (sinon juste avec une fstring c’est la valeur évaluée au moment de la génération de la migration qui se retrouve définitivement utilisée).
Edit: j’ai aussi remplacé le validateur custom par une combinaison du MinValueValidator et du MaxValueValidator de Django, qui font très bien l’affaire (et qui disposent déjà de la sérialisation nécessaire à leur déclaration dans la migration).
313de2a763toea5bfc9c7cea5bfc9c7cto7f5b49fb587f5b49fb58tof3e57d5089@ -235,1 +236,4 @@def sms_code_duration_help_text():return _(f'Time (in seconds, between 60 and 3600) after which SMS codes expire. Default is {settings.SMS_CODE_DURATION}.'Ah ouais carrément. Trop fort les f-strings.