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

Revert "Automated SimpleSAMLphp htaccess patch." #605

Merged
merged 1 commit into from
Oct 26, 2016

Conversation

grasmash
Copy link
Contributor

Reverts #603

@grasmash grasmash merged commit eaf0bc3 into 8.x Oct 26, 2016
* Copies configuration files to `${project.root}/simplesamlphp`
* Adds a simplesamlphp property to project.yml
* Creates a symbolic link in the docroot to the web accessible directory of the simplesamlphp library.
* Adds a settings.php file to the project's default settings directory.


1. Edit `docroot/.htaccess` to include the following 2 lines. Note: the only 2 lines that need to be added are the lines marked with "+" signs.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danepowell @grasmash Are there any accompanying changes in Acquia's stock VCL that are needed to make this setup work and should be documented here as well?

Copy link
Contributor

@danepowell danepowell Oct 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there something in particular you think would need to change? We've done SAML on multiple projects now using this setup and haven't had to adjust the default ACE VCL.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danepowell I believe you wouldn't need to modify the ACE VCL for SP (Drupal) initiated SSO. Do you know if anyone has tried this SimpleSAMLphp setup with IDP (Unsolicited) SSO?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seeing this now. We modified our VCL to pass paths beginning with /simplesaml since we do IDP SSO. Has been working for us.

@grasmash grasmash added the Enhancement A feature or feature request label Nov 1, 2016
@grasmash grasmash deleted the revert-603-feature/saml-auto branch November 16, 2016 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants