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

Updating single sign on documentation #797

Closed
wants to merge 1 commit into from
Closed

Updating single sign on documentation #797

wants to merge 1 commit into from

Conversation

tic984
Copy link

@tic984 tic984 commented May 6, 2024

Updating single sign on documentation with information on explicitly setting the domain via configuration.

Q A
License MIT

What's in this PR?

This PR introduces an amendment to the documentation regarding Single Sign-On (SSO) configuration in the Sulu project.

Why?

The existing documentation lacked clarity regarding the configuration option for specifying the domain. This PR addresses this issue by providing explicit instructions on how to configure the domain key, which is essential for seamless integration with Single Sign-On providers. Additionally, it includes a note highlighting the importance of specifying the domain key to avoid potential normalization issues.

I enountered the issue while following the sample in the documentation and provided a domain with "-" in it.

@alexander-schranz
Copy link
Member

alexander-schranz commented May 13, 2024

@tic984 Thx for the pull request but this is more a bug then a documentation part. This will be fixed in: sulu/sulu#7421. Still thank you for your pull request.

@alexander-schranz
Copy link
Member

Thank you still for your pr. added you as a co authored of sulu/sulu#7421 which should fix the issue with - in future.

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 this pull request may close these issues.

2 participants