Projet

Général

Profil

Bug #7822

CAS URLs : permit call without trailing slash

Ajouté par Benjamin Renard il y a plus de 8 ans. Mis à jour il y a plus de 6 ans.

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

Révision 1b5a1d28 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 8 ans

authentic2_idp_cas: accept URL without the trailing slashes to conform to the specification (fixes #7822)

Historique

#1

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)

#2

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
#3

Mis à jour par Benjamin Renard il y a plus de 8 ans

Benjamin Dauvergne a écrit :

Appliqué par commit authentic2|1b5a1d28d188cbf7fa67468c0e53644747fddeb6.

Merci !

#4

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

  • Assigné à changé de Gestion d'identité à Benjamin Dauvergne
#5

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

  • Version cible mis à 2.2.0
#6

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

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

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF