Bug #6015: Error raised on when oauth2 webservice called by its slug - Authentic 2 - Redmine Entr’ouvert

Bug #6015

Error raised on when oauth2 webservice called by its slug

Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 9 ans. Mis à jour il y a plus de 9 ans.

Statut:

Fermé

Priorité:

Normal

Assigné à:

Serghei Mihai (congés, retour 15/05)

Catégorie:

Version cible:

Début:

27 novembre 2014

Echéance:

% réalisé:

100%

Temps estimé:

Patch proposed:

Oui

Planning:

Description

The webservice view allows to call a oauth2 webservice by its slug, but it raises an exception when it's done.

Here is an example:

ValueError at /idp/oauth2/ws-proxy/agglo/
    invalid literal for int() with base 10: 'agglo'

Fichiers

0001-handling-more-properly-cases-where-webservice-is-cal.patch (1002 octets) 0001-handling-more-properly-cases-where-webservice-is-cal.patch		Serghei Mihai (congés, retour 15/05), 27 novembre 2014 07:55
0001-Do-not-crash-on-an-invalid-WebService-id-or-an-unkno.patch (1,14 ko) 0001-Do-not-crash-on-an-invalid-WebService-id-or-an-unkno.patch		Benjamin Dauvergne, 04 décembre 2014 00:23

Révisions associées

Révision 9b7d3cec (diff)
Ajouté par Benjamin Dauvergne il y a plus de 9 ans

Do not crash on an invalid WebService id or an unknown WebService slug

fixes #6015

Historique

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 9 ans

Fichier 0001-handling-more-properly-cases-where-webservice-is-cal.patch 0001-handling-more-properly-cases-where-webservice-is-cal.patch ajouté
Patch proposed changé de Non à Oui

Trying to cast the webservice id and handling the exception

Mis à jour par Frédéric Péters il y a plus de 9 ans

In my opinion there's a bit more to do to clean this function, it will for example still fail unexpectedly if an invalid slug is given. (but honestly I am not a fan of this fallback on slug pattern).