-
Notifications
You must be signed in to change notification settings - Fork 396
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
acquia_config.php for SimpleSAMLphp does not work on Acquia Cloud ODE. #3437
Comments
@wu-edward, I am curious how would you test SSO without a constant domain name? Every-time that you spin up a new CDE wouldn't you need to exchange new metadata with the domain? I ask because if you have a clever solution I would LOVE to steal it! |
@aweingarten We spun up a CDE specifically for a PoC we're building that requires SSO, so unfortunately no cleverness involved. |
Can you try replacing If that doesn't work, you may need to debug this a little more. Unfortunately I don't have access to an environment with SAML and CDEs to test this. @lcatlett removed the code you are referring to in #2953, she might have insight as well |
if you use a relative path for the saml config then it can work on any environment - it is when baseurl and other config is hardcoded (and there may be some valid reasons) that it appears to 'work' only on one environment / site / hosting provider. It is more likely that this is the same issue I called out in #3492 (comment) |
@danepowell I took your suggestion and set |
In the
acquia_config.php
included in BLT, the database credentials are extracted in lines 95-97:The hostname of the database is then looked up on line 106-107:
On the ODE I want to run SSO tests on, $creds['db_cluster_id'] is
null
, so the host is empty, which causes an exception. I was able to fix this by hardcoding the database host for the specific ODE environment it was on.But since https://support.acquia.com/hc/en-us/articles/360008830834 implies that ODEs should work, this code may need to be updated to get the DB host for ODEs.
The text was updated successfully, but these errors were encountered: