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

support SSO with SAML #138

Closed
wants to merge 3 commits into from
Closed

support SSO with SAML #138

wants to merge 3 commits into from

Conversation

napat1412
Copy link

I upgrade keyrock to support Single Sign On. User can sign-in by using idp with SAML. When admin enable external_user_sso (this feature), user can sign-in with with SAML to setup local password in SQL database. After that user has 2 way to sign-in. (1 sign-in with local password 2 sign-in with SAML).

I use keycloak as idp, and setup 2 mappers for SAML client.

  1. Name: username, Type: User Property, Property: username
  2. Name: email, Type: User Property, Property: email

I appreciate if you merge this code or rewrite it to keyrock because i really need this feature (external user can sign-in with SAML or open-id-conect).

root added 3 commits February 25, 2020 19:19
	modified:   config.js.template
	modified:   controllers/web/index.js
	modified:   controllers/web/sessions.js
	modified:   controllers/web/settings.js
	new file:   controllers/web/sso.js
	modified:   controllers/web/users.js
	modified:   package-lock.json
	modified:   package.json
	modified:   routes/web/authenticate.js
	modified:   routes/web/index.js
	modified:   views/auth/_login.ejs
	modified:   views/settings/settings.ejs
	modified:   doc/installation_and_administration_guide/configuration.md
	modified:   views/settings/settings.ejs
@napat1412
Copy link
Author

move to #139

@napat1412 napat1412 closed this Feb 26, 2020
@napat1412 napat1412 deleted the sso_saml branch February 26, 2020 07:53
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.

None yet

1 participant