Skip to content

Commit

Permalink
OpenID Connect Support (#423)
Browse files Browse the repository at this point in the history
* Support OIDC

* Rework to use new structs from operator-rs

* charts

* handle tls settings

* charts

* new operator-rs

* update operator-rs

* fix: Move oidcApiPath to the correct location

* chore: update operator-rs

* feat: erorr out if superset is configured for oidc+keycloak with a principal_claim other than preferred_username

* update operator-rs

* bump

* Update operator-rs

* fix tests

* Make authentication configuration optional

* Rename SupersetAuthenticationConfigResolved to SupersetClientAuthenticationDetailsResolved

* Improve the data type of the authentication configuration

* Regenerate charts

* Revise the resolution of OIDC authentication details

* Add OIDC test

* Use TLS for Keycloak in the OIDC test

* Check the user info in the OIDC test

* Fix linter warnings

* Fix secret scope in the OIDC test

* Refactor the check that TLS verification cannot be disabled

* Allow multiple OIDC providers

* Add documentation for OIDC in Superset

* Use main branch of operator-rs

* Fix spelling

* Add comments

* Move the creation of EnvVars into a separate function

* Update docs/modules/superset/pages/usage-guide/security.adoc

Co-authored-by: Razvan-Daniel Mihai <[email protected]>

* Update docs/modules/superset/pages/usage-guide/security.adoc

Co-authored-by: Razvan-Daniel Mihai <[email protected]>

* Update docs/modules/superset/pages/usage-guide/security.adoc

Co-authored-by: Razvan-Daniel Mihai <[email protected]>

* Upgrade operator-rs to version 0.64.0

* Update changelog

* Fix external link icons in the documentation

---------

Co-authored-by: Siegfried Weber <[email protected]>
Co-authored-by: Nick Larsen <[email protected]>
Co-authored-by: Razvan-Daniel Mihai <[email protected]>
  • Loading branch information
4 people authored Jan 31, 2024
1 parent a0212da commit 30f76ca
Show file tree
Hide file tree
Showing 30 changed files with 2,215 additions and 610 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,13 @@

- Improved CRD docs ([#431]).
- Helm: support labels in values.yaml ([#448]).
- Add support for OpenID Connect ([#423]).

### Fixed

- BREAKING: Fixed various issues in the CRD structure. `clusterConfig.credentialsSecret` is now mandatory ([#429]).

[#423]: https://github.com/stackabletech/superset-operator/pull/423
[#429]: https://github.com/stackabletech/superset-operator/pull/429
[#431]: https://github.com/stackabletech/superset-operator/pull/431
[#448]: https://github.com/stackabletech/superset-operator/pull/448
Expand Down
Loading

0 comments on commit 30f76ca

Please sign in to comment.