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

Oidc provider uses storage type also for oidc client configuration #1269

Merged
merged 15 commits into from
Apr 24, 2020

Conversation

maxmarkus
Copy link
Contributor

@maxmarkus maxmarkus commented Apr 15, 2020

OIDC Provider now respects the auth.storage setting and uses the same storage not only for Luigi but also for the oidc client library.

How to test with e2e application:

  • Run oidc mock server from ./scripts/oidc-mockserver with docker-compose up
  • Set auth.use to openIdConnect
  • Set the auth.storage type you want to test

Test all three cases.

Fixes #1190

@maxmarkus maxmarkus added enhancement New feature or request auth authorization/authentication labels Apr 15, 2020
@maxmarkus maxmarkus added this to the Sprint 10 milestone Apr 15, 2020
@maxmarkus maxmarkus changed the title Oidc provider uses storage type also in oidc client Oidc provider uses storage type also for oidc client configuration Apr 15, 2020
@JohannesDoberer JohannesDoberer self-assigned this Apr 16, 2020
@zarkosimic zarkosimic self-assigned this Apr 21, 2020
Copy link
Contributor

@JohannesDoberer JohannesDoberer left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@zarkosimic zarkosimic left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@maxmarkus maxmarkus merged commit 767a638 into SAP:master Apr 24, 2020
@maxmarkus maxmarkus deleted the 1190-oidc-align-storage-types branch April 24, 2020 09:50
This was referenced Apr 24, 2020
JohannesDoberer added a commit to JohannesDoberer/luigi that referenced this pull request Apr 27, 2020
* master:
  updated luigi version in fiddle and docs website (SAP#1298)
  Refactor Svelte example app (SAP#1270)
  Release v1.1.0 (SAP#1297)
  update fundamental styles library version in angular example 0.8.1 (SAP#1245)
  Fix semicollapsible issue (SAP#1292)
  Translate tooltip text in semi collapsed mode and add title attributes to entries in left nav (SAP#1283)
  Oidc provider uses storage type also for oidc client configuration (SAP#1269)
stanleychh pushed a commit to stanleychh/luigi that referenced this pull request Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth authorization/authentication enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OIDC provider: align userStore config with auth.storage setting
3 participants