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

Add public SignOut method to CasAuthentication class #37

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

rdev5
Copy link
Contributor

@rdev5 rdev5 commented Jan 15, 2016

By default, Request Validation in ASP.NET kicks in upon receiving a single sign-out request from CAS due to the nature of its POSTed content (i.e. XML).

This pull requests provides utility for developers to call CasAuthentication.SignOut(); which will not only invoke FormsAuthentication.SignOut (as a native replacement to ClearAuthCookies()) but also clear the user's service ticket from cache, which is currently not being done.

By default, [Request Validation in ASP.NET](https://msdn.microsoft.com/en-us/library/hh882339(v=vs.110).aspx) kicks in upon receiving a single sign-out request from CAS due to the nature of its POSTed content (i.e. XML).

This pull requests provides utility for developers to call `CasAuthentication.SignOut();` which will not only invoke `FormsAuthentication.SignOut` (as a native replacement to `ClearAuthCookies()`) but also clear the user's service ticket from cache, which is currently not being done.
@serac
Copy link
Contributor

serac commented Jan 18, 2016

The documentation you cited mentions HTML content specifically, but the payload of the single sign-out request is XML. Is XML specifically considered "potentially dangerous content," thus causing requests to be rejected? Just trying to follow your logic on the problem here.

@phantomtypist phantomtypist changed the base branch from master to develop August 17, 2017 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants