Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Access service application always redirect cas #4

Open
Delvius opened this issue Jan 13, 2016 · 5 comments
Open

Access service application always redirect cas #4

Delvius opened this issue Jan 13, 2016 · 5 comments
Assignees
Labels

Comments

@Delvius
Copy link

Delvius commented Jan 13, 2016

Hi,
i have a trouble with this plugin. My application always redirect me to the cas authentication page (at the first connection) even if i already have a cas ticket.

My config :

var cas = new CASAuthentication({
cas_url : 'XXXX',
service_url : 'XXXX',
cas_version : '3.0',
renew : false,
is_dev_mode : false,
dev_mode_user : '',
dev_mode_info : {},
session_name : 'cas_user',
session_info : 'cas_userinfo',
destroy_session : false

});

app.get('/',cas.bounce_redirect, controllers.index);

Any ideas ?

@matthewvalimaki
Copy link

@Delvius per official CAS protocol documentation https://jasig.github.io/cas/development/protocol/CAS-Protocol-Specification.html#parameters:

renew [OPTIONAL] - if this parameter is set, single sign-on will be bypassed. In this case, CAS will require the client to present credentials regardless of the existence of a single sign-on session with CAS.

Many comments around the nets suggest that "renew=false" could be used. Either protocol has changed or implementation has been broken. Either way cas-authentication should not add it to the request unless something else than false.

matthewvalimaki added a commit to matthewvalimaki/cas-authentication that referenced this issue Apr 11, 2016
@matthewvalimaki
Copy link

@Delvius I've forked and published my version if you want to give it a try. See https://www.npmjs.com/package/r-cas-authentication.

@sharumpe
Copy link

I'm glad I looked -- I was going to create a similar pull request for this. Thought it was just my CAS configuration or something.

@dgeo
Copy link

dgeo commented Feb 2, 2017

@matthewvalimaki thank you !

@kylepixel thank you for sharing at first ! now: pass the hand or support ?

@kayleecodes1
Copy link
Owner

Hey, all! I've been pretty inactive on my Github projects recently. I originally published this to NPM to quickly get it into our application at my job but I'm planning on revisiting this and handling all of the requests / pull requests. Sorry about that.

@kayleecodes1 kayleecodes1 self-assigned this Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants