Bug #7822
CAS URLs : permit call without trailing slash
Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
authentic2-idp-cas
Version cible:
Début:
10 juillet 2015
Echéance:
% réalisé:
100%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
When Authentic2 acting as CAS server, well-known URLs (/login, /serviceValidate, ...) required a trailing slash. Majority of CAS client libraries I known, does not add this trailing slash and simply call, for instance :
https://cas.domain.tld/cas/login?service=~~~ https://cas.domain.tld/cas/serviceValidate?service=~~~&ticket=~~
To handle this request, I mean you could simply made the trailing slash optional. I propose to do this in authentic2_idp_cas/urls.py :
urlpatterns = patterns('', url('^login/?$', views.login, name='a2-idp-cas-login'), url('^continue/?$', views._continue, name='a2-idp-cas-continue'), url('^validate/?$', views.validate, name='a2-idp-cas-validate'), url('^serviceValidate/?$', views.service_validate, name='a2-idp-cas-service-validate'), url('^logout/?$', views.logout, name='a2-idp-cas-logout'), url('^proxy/?$', views.proxy, name='a2-idp-cas-proxy'), url('^proxyValidate/?$', views.proxy_validate, name='a2-idp-cas-proxy-validate'), )
I only add this character ? after the trailing slashes in URLs pattern. This work well with phpCAS library.
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Could you provide a git patch ? I'll integrate it immediately. (from git format-patch)
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|1b5a1d28d188cbf7fa67468c0e53644747fddeb6.
Mis à jour par Benjamin Renard il y a plus de 8 ans
Benjamin Dauvergne a écrit :
Appliqué par commit authentic2|1b5a1d28d188cbf7fa67468c0e53644747fddeb6.
Merci !
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Assigné à changé de Gestion d'identité à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Solution déployée à Fermé
authentic2_idp_cas: accept URL without the trailing slashes to conform to the specification (fixes #7822)