Project

General

Profile

Bug #42659

api particulier, 'NoneType' object has no attribute 'status_code' lors de timeout de l'api

Added by sentry io about 2 months ago. Updated 24 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
08 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/2063/

AttributeError: 'NoneType' object has no attribute 'status_code'
  File "passerelle/utils/jsonresponse.py", line 128, in api
    resp = f(*args, **kwargs)
  File "passerelle/views.py", line 475, in perform
    result = self.endpoint(request, **params)
  File "passerelle/apps/api_particulier/models.py", line 144, in impots_svair
    }, user=user)
  File "passerelle/apps/api_particulier/models.py", line 85, in get
    (self.platform, response.status_code),

Error occurred while processing request

0001-api_particulier-fix-connection-error-42659.patch View (4.01 KB) Lauréline Guerin, 11 May 2020 09:24 AM

0001-api_particulier-fix-connection-error-42659.patch View (3.96 KB) Lauréline Guerin, 12 May 2020 10:30 AM

0001-api_particulier-fix-connection-error-42659.patch View (5.6 KB) Lauréline Guerin, 11 Jun 2020 02:56 PM

Associated revisions

Revision 2ff16b61 (diff)
Added by Lauréline Guérin 24 days ago

api_particulier: fix connection error (#42659)

History

#1 Updated by Frédéric Péters about 2 months ago

  • Project changed from Suivi des traces to Passerelle
  • Subject changed from AttributeError: 'NoneType' object has no attribute 'status_code' to api particulier, 'NoneType' object has no attribute 'status_code' lors de timeout de l'api

#2 Updated by Lauréline Guerin about 2 months ago

  • Assignee set to Lauréline Guerin

#3 Updated by Lauréline Guerin about 2 months ago

#4 Updated by Nicolas Roche about 2 months ago

Juste pour comprendre, pourquoi tu ne pointe pas sur conversions.py pour importer exception_to_text ?

from passerelle.utils.jsonresponse import APIError, exception_to_text

#5 Updated by Lauréline Guerin about 2 months ago

mauvais copier/coller, merci :)

#7 Updated by Valentin Deniaud 27 days ago

  • Status changed from Solution proposée to Solution validée

Juste que je ne toucherais pas au paramètre data de l'APIError, peut-être qu'à un endroit il y a un wcs qui regarde ce qu'il y a dans response_data_data_platform, qui sait.

#8 Updated by Benjamin Dauvergne 27 days ago

Valentin Deniaud a écrit :

Juste que je ne toucherais pas au paramètre data de l'APIError, peut-être qu'à un endroit il y a un wcs qui regarde ce qu'il y a dans response_data_data_platform, qui sait.

Pareil je ne vois pas le besoin de diminuer l'information disponible, à la rigueur l’accroître avec un code stable utilisable dans des conditions ça j'aime bien.

#9 Updated by Valentin Deniaud 27 days ago

Benjamin Dauvergne a écrit :

à la rigueur l’accroître avec un code stable utilisable dans des conditions ça j'aime bien.

Il y a déjà un patch qui fait ça dans #38721, pas besoin d'y toucher ici.

#10 Updated by Benjamin Dauvergne 27 days ago

Valentin Deniaud a écrit :

Il y a déjà un patch qui fait ça dans #38721, pas besoin d'y toucher ici.

Oublié, donc reste juste à ne pas perdre le platform au passage.

#11 Updated by Lauréline Guerin 25 days ago

J'avais copié/collé la façon de faire dans api_entreprise. Mauvaise idée apparemment :)

(note: j'ai supprimé la redéfinition de vector dans le test test_error, c'est les mêmes valeurs à chaque fois)

#12 Updated by Valentin Deniaud 25 days ago

  • Status changed from Solution proposée to Solution validée

#13 Updated by Lauréline Guerin 24 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 2ff16b6105780ac6774dc962e9b4c038192e0208
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon May 11 09:21:48 2020 +0200

    api_particulier: fix connection error (#42659)

#14 Updated by Frédéric Péters 24 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF