Skip to content

Support simultaneous standard & SSO login#163

Merged
shawn-h-park merged 2 commits intoSSO-Supportfrom
Simult-Login
Oct 13, 2021
Merged

Support simultaneous standard & SSO login#163
shawn-h-park merged 2 commits intoSSO-Supportfrom
Simult-Login

Conversation

@shawn-h-park
Copy link
Contributor

@shawn-h-park shawn-h-park commented Oct 13, 2021

Multi-login support: - Be able to land on login page and choose whether to use standard login or SSO login

Note:

  • Merging to SSO-Support branch to release together

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

Copy link
Collaborator

@kevin-mcgoldrick kevin-mcgoldrick left a comment

Choose a reason for hiding this comment

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

LGTM

@shawn-h-park shawn-h-park merged commit c9953f2 into SSO-Support Oct 13, 2021
shawn-h-park added a commit that referenced this pull request Oct 13, 2021
* Add support to get user by email

* OIDC SSO Support

* Remove Http Client Injector given Static builder

* Add initial exception handling & logs to SSO Flow

* Add initial auth request redirect

* Handle initial redirect from authorization server

* LoginFilterTest unit test

* (1) Handle redirect TO & FROM Authorization server + (2) Absence of SSO Config defaults to login

* Adjust ordering of filter logic to prevent redirect cycle

* Handle SSO User Credentials

* Support simultaneous standard & SSO login (#163)

* Support simultaneous standard & sso login

* (1) Switch SSO button order to be on top + (2) Render upon config present
@kevin-mcgoldrick kevin-mcgoldrick deleted the Simult-Login branch December 9, 2021 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants