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

Single Sign-on with Pantheon Doc Update #8749

Closed
v0jpantheon opened this issue Oct 17, 2023 · 0 comments · Fixed by #8767
Closed

Single Sign-on with Pantheon Doc Update #8749

v0jpantheon opened this issue Oct 17, 2023 · 0 comments · Fixed by #8767

Comments

@v0jpantheon
Copy link
Contributor

Re: Single Sign-on with Pantheon

Priority: Medium (choose one, remove the other options)

Issue Description:

As of simplesamlphp:v2.0.6, the config-template is no longer available, so the steps "3. Create your site-specific config file:" under the "Composer Method" tab is no long applicable and needs to be updated.

Suggested Resolution

The path vendor/simplesamlphp/simplesamlphp/config now has /config/*.dist file that they can copy separately so that the config file will not be replaced when the composer installs again.

New *.dist files:
/config/acl.php.dist
/config/authsources.php.dist
/config/config.php.dist

Updates to step 3:

mkdir -p private/simplesamlphp
mv vendor/simplesamlphp/simplesamlphp/config private/simplesamlphp/config
cp private/simplesamlphp/config/config.php.dist private/simplesamlphp/config/config.php
cp private/simplesamlphp/config/authsources.php.dist private/simplesamlphp/config/authsources.php

Remove Steps 5, since this will be done by the composer on step 6 anyway.

For step 6:
No need to remove any files anymore, we only need symlinks.

"scripts": {
   "post-update-cmd": [
       "ln -sf ../../../private/simplesamlphp/config vendor/simplesamlphp/simplesamlphp/config"
   ],
   "post-install-cmd": [
       "ln -sf ../../../private/simplesamlphp/config vendor/simplesamlphp/simplesamlphp/config"
   ]
},
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 a pull request may close this issue.

1 participant