Bug #61222
ValueError: year 275812 is out of range
0%
Description
https://sentry.entrouvert.org/entrouvert/gplexpert/issues/54394/
ValueError: year 275812 is out of range
(8 additional frame(s) were not displayed)
...
File "wcs/sql.py", line 3848, in migrate
migrate_legacy_tokens()
File "wcs/sql.py", line 1220, in migrate_legacy_tokens
token = wcs.qommon.tokens.Token.get(token_id)
File "wcs/qommon/storage.py", line 498, in get
filename, ignore_errors=ignore_errors, ignore_migration=ignore_migration, **kwargs
File "wcs/qommon/storage.py", line 631, in get_filename
o.migrate()
File "wcs/qommon/tokens.py", line 63, in migrate
self.expiration = make_aware(datetime.datetime.fromtimestamp(self.expiration))
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
Contenu du token :
{'id': '2jfUkpAHG6nAqtpm2YwOonjb0nzg4jVutv9TmbzZ6hANrYp2Kq1aypIDWV9Gjx6j',
'expiration': 8641635948419.479,
'type': 'action',
'context': {
'form_slug': 'reservation-des-animations-dans-les-parcs-naturels-departementaux',
'form_number_raw': '82',
'action_id': 'desistement',
'label': 'annuler ma réservation'}
}
Dans un workflow :
Vous pouvez annuler à tout moment en cliquant sur {% action_button "desistement" label="annuler ma réservation" delay=99999999 %}
Va falloir poser des limites.
(en attendant j'ai viré tous les jetons, mais faut gérer ça dans la migration avant le passage en prod)
Mis à jour par Thomas Noël il y a environ 2 ans
Je propose deux semaines, et ça me semble déjà bien généreux. C'est de l'action par mail, prévue pour ne pas être longtemps attendue.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-tokens-limit-expiration-to-1-year-61222.patch 0001-tokens-limit-expiration-to-1-year-61222.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Dans le patch et dans la branche j'avais tapé 365 jours, ça m'irait de le passer ainsi éviter l'attente d'une discussion, juste pour être sûr que ça sera bien présent dans la mise à jour de prod.
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Dans le patch et dans la branche j'avais tapé 365 jours, ça m'irait de le passer ainsi éviter l'attente d'une discussion, juste pour être sûr que ça sera bien présent dans la mise à jour de prod.
Allons-y ainsi, un an c'est très trèèèès large.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b8e07bd5c89ee9847e98c62874b5900f26d7f484
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat Jan 29 10:57:06 2022 +0100
tokens: limit expiration to 1 year (#61222)
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tokens: limit expiration to 1 year (#61222)