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

Flexible NameID #111

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Flexible NameID #111

wants to merge 6 commits into from

Conversation

upwebdesign
Copy link
Contributor

Changes made for NameID for SAML assertions and metadata XML page.

  • Add two new options to the config file for setting NameID format (original default value is used if not set) and an option to turn off NameID altogether in assertions.
  • Update metadata XML file to use dynamic NameID format from the config, if set.

This code has yet to be tested. Unfortunately, time is tight. If anyone has the time to give this a try, please do!

Add an option in config to set the NameID format.
Add a config option to set the NameID format for the subject. Default to email.
Update metadata XML file to use dynamic NameID format from the config
Add a config option to define whether or not we want to use NameID in assertions.
Add a check to see if we want to use the NameID in the assertion according to the config value.
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.

1 participant