Projet

Général

Profil

Development #66974

AssertionError

Ajouté par Sentry Io il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 juillet 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/nfrance/issues/62129/

AssertionError: 
  File "passerelle/utils/jsonresponse.py", line 129, in api
    resp = f(*args, **kwargs)
  File "passerelle/views.py", line 551, in perform
    result = self.endpoint(request, **params)
  File "passerelle/contrib/toulouse_axel/models.py", line 755, in invoices
    invoices_data = self.get_invoices(regie_id=regie_id, name_id=NameID)
  File "passerelle/contrib/toulouse_axel/models.py", line 679, in get_invoices
    assert name_id or dui

Error occurred while processing request

Fichiers

Révisions associées

Révision ed9a4a29 (diff)
Ajouté par Corentin Séchet il y a presque 2 ans

toulouse_axel: Forbid blank NameID for invoices endpoint (#66974)

Historique

#1

Mis à jour par Corentin Séchet il y a presque 2 ans

  • Projet changé de Suivi des traces à Passerelle
  • Assigné à mis à Lauréline Guérin

Je ne sais pas dans quelle mesure il faut gérer name_id qui est une chaîne vide, plutôt que l'asserter, donc je pose ça là.

#2

Mis à jour par Lauréline Guérin il y a presque 2 ans

  • Assigné à changé de Lauréline Guérin à Corentin Séchet

Oui, c'est null l'assertion en plein milieu du code. Tu veux proposer un patch pour renvoyer une erreur propre ?

#3

Mis à jour par Corentin Séchet il y a presque 2 ans

#4

Mis à jour par Frédéric Péters il y a presque 2 ans

  • Statut changé de Solution proposée à Solution validée
#5

Mis à jour par Corentin Séchet il y a presque 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit ed9a4a29863af8d17bffbaf218ad11b4eaef8ef4
Author: Corentin Séchet <csechet@entrouvert.com>
Date:   Tue Jul 5 11:21:44 2022 +0200

    toulouse_axel: Forbid blank NameID for invoices endpoint (#66974)
#6

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF