Project

General

Profile

Development #39911

rest_authentication : lever des exceptions spécifiques en cas d'échec

Added by Benjamin Dauvergne 5 months ago. Updated 3 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
14 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

DRF transformer toutes les exceptions AuthenticationFailed en PermissionDenied et sans {'err': 1} qu'on aime tant.

On voudrait lever des PublikAuthenticationFailed('invalid-signature' / 'user-not-found'}

Produisant du

401 AuthenticationFailed
Content-Type: application/json

{'err': 'invalid-signature'} ou {'err': 'user-not-found'}

0001-rest_authentication-raise-APIError-for-signature-err.patch View (4.74 KB) Benjamin Dauvergne, 09 Mar 2020 12:03 PM

0001-rest_authentication-raise-APIError-for-signature-err.patch View (8 KB) Benjamin Dauvergne, 09 Mar 2020 10:21 PM


Related issues

Related to Combo - Development #35352: fargo, ne pas logger d'erreur de GET sur une 403 Solution déployée 10 Aug 2019

Associated revisions

Revision 9225efdb (diff)
Added by Benjamin Dauvergne 4 months ago

rest_authentication: raise APIError for signature errors (#39911)

History

#1 Updated by Benjamin Dauvergne 5 months ago

#2 Updated by Benjamin Dauvergne 5 months ago

  • Assignee set to Benjamin Dauvergne

#3 Updated by Benjamin Dauvergne 4 months ago

#4 Updated by Thomas Noël 4 months ago

Ca m'ennuie qu'on ait pas un test qui vérifie qu'on reçoit un 401 avec {err:quelque-chose} ... On peut ou pas avec RequestFactory ? (je connais mal)

#5 Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution proposée to En cours

Thomas Noël a écrit :

Ca m'ennuie qu'on ait pas un test qui vérifie qu'on reçoit un 401 avec {err:quelque-chose} ... On peut ou pas avec RequestFactory ? (je connais mal)

Oui je dois pouvoir faire ça.

#6 Updated by Benjamin Dauvergne 4 months ago

Voilà.

#7 Updated by Thomas Noël 4 months ago

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

Naïce.

#8 Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 9225efdb96b087f919518fdf4e3c21441eb90d16
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Mar 9 12:02:55 2020 +0100

    rest_authentication: raise APIError for signature errors (#39911)

#9 Updated by Frédéric Péters 3 months ago

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

Also available in: Atom PDF