Development #52444
crash sur /api/users/ quand une erreur qui n'existe pas est passée (NonExistentTimeError: 2021-03-28 02:00:00)
Début:
29 mars 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/41352/
NonExistentTimeError: 2021-03-28 02:00:00 (24 additional frame(s) were not displayed) ... File "django_filters/fields.py", line 123, in strptime return handle_timezone(parsed) File "django_filters/utils.py", line 149, in handle_timezone return make_aware(value, timezone.get_current_timezone(), is_dst) File "django_filters/compat.py", line 69, in make_aware return make_aware_orig(value, timezone, is_dst) File "django/utils/timezone.py", line 285, in make_aware return timezone.localize(value, is_dst=is_dst) File "pytz/tzinfo.py", line 341, in localize raise NonExistentTimeError(dt)
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 3 ans
- Projet changé de Suivi des traces à Authentic 2
- Sujet changé de NonExistentTimeError: 2021-03-28 02:00:00 à crash sur /api/users/ quand une erreur qui n'existe pas est passée (NonExistentTimeError: 2021-03-28 02:00:00)
.../api/users/?modified__gte=2021-03-28T02%3A00%3A00
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Priorité changé de Normal à Bas
Je crois qu'on peut vivre avec, coté DRF c'est corrigé en 3.13 (https://github.com/encode/django-rest-framework/issues/4986) mais pas dans le sens qui nous plairait, ça devient une erreur propre, il n'y a pas de tentative de 'disambiguation', on est sensé envoyé une date avec un offset de timezone à une API Web.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Fichier 0001-api_views-handle-both-kind-of-TimeError-52444.patch 0001-api_views-handle-both-kind-of-TimeError-52444.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Assigné à
Benjamin Dauvergnesupprimé
Oublié que je traitais déjà une partie du problème.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Benjamin Dauvergne
Mis à jour par Paul Marillonnet il y a environ 3 ans
(Rebase nécessaire je pense. C’est pas valide du point de vue de isort.)
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9be334320d49fa8092fb73464ac4a30d82442869 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Mar 29 15:09:33 2021 +0200 api_views: handle both kind of TimeError (#52444)
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Paul Marillonnet a écrit :
(Rebase nécessaire je pense. C’est pas valide du point de vue de isort.)
Je l'ai fait avant de pousser.
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api_views: handle both kind of TimeError (#52444)