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

Fixed SpnegoWebflowConfigurer to use the evaluateClientRequest as documented #3411

Merged
merged 1 commit into from Jul 16, 2018
Merged

Conversation

miller79
Copy link
Contributor

Changes include the following:

  • Added conditional logic when mixedModeAuthentication is false to use
    startSpnegoAuthenticate flow. Otherwise it will use
    evaluateClientRequest.
  • Modified createEvaluateSpnegoClientAction to be equivlent to the other
    Spnego actions on failure which is
    CasWebflowConstants.STATE_ID_VIEW_LOGIN_FORM

Since this change fixes the documentation listed out here, this change re-enables the functionality which may cause unexpected results to those using the defaults. Some of those properties are the following:

  • cas.authn.spnego.hostNameClientActionStrategy
  • cas.authn.spnego.hostNamePatternString
  • cas.authn.spnego.mixedModeAuthentication
  • cas.authn.spnego.ipsToCheckPattern

I have done my best to ensure as few changes are affected as possible.

documented.  Changes include the following:
- Added conditional logic when mixedModeAuthentication is false to use
startSpnegoAuthenticate flow.  Otherwise it will use
evaluateClientRequest.
- Modified createEvaluateSpnegoClientAction to be equivlent to the other
Spnego actions on failure which is
CasWebflowConstants.STATE_ID_VIEW_LOGIN_FORM
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Anthony Lofton seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mmoayyed mmoayyed merged commit e8edcf3 into apereo:5.2.x Jul 16, 2018
@mmoayyed mmoayyed added this to the 5.2.7 milestone Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants