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

SSO not working with SAML2 and ADFS #6524

Open
sonar98 opened this issue Nov 23, 2018 · 0 comments
Open

SSO not working with SAML2 and ADFS #6524

sonar98 opened this issue Nov 23, 2018 · 0 comments
Labels
Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type: Bug Bugs within the core SuiteCRM codebase

Comments

@sonar98
Copy link

sonar98 commented Nov 23, 2018

Because a standard setting in Onelogin SSO is not working when using SAML2 in combination with ADFS.
Users get a login screen from ADFS when opening SuiteCRM. Other applications using the same ADFS server are logged in automatically.

According to an issue at the github of onelogin this is a setting in de config of onelogin that defaults to the wrong value.
SAML-Toolkits/php-saml#62

It is a simple change in the settings.php
`\modules\Users\authentication\SAML2Authenticate\lib\onelogin\php-saml\lib\Saml2\

Original:
if (!isset($this->_security['requestedAuthnContext'])) {
$this->_security['requestedAuthnContext'] = true;
}

Working:
if (!isset($this->_security['requestedAuthnContext'])) {
$this->_security['requestedAuthnContext'] = false;
}
`

@Dillon-Brown Dillon-Brown added Type: Bug Bugs within the core SuiteCRM codebase Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds labels Dec 3, 2018
@jack7anderson7 jack7anderson7 added the PR:Community Contribution These are contribution made by the community label Dec 7, 2018
@Dillon-Brown Dillon-Brown removed PR:Community Contribution These are contribution made by the community labels Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type: Bug Bugs within the core SuiteCRM codebase
Projects
None yet
Development

No branches or pull requests

3 participants