Skip to content

Commit c851f51

Browse files
sbernauerTechassi
andauthored
fix: Construction of OIDC endpoint when rootPath has no trailing slash (#718)
* refactor: Move oidc config creation to oidc.rs * Use new op-rs * nix files * changelog * Revert "nix files" This reverts commit 739d417. * new op-rs version * changelog * use op-rs 0.82.0 * remove uneeded error * Add patch option * nixfiles * Apply suggestions from code review Co-authored-by: Techassi <git@techassi.dev> * fix compilation * calculation -> construction * Rename fn to add_oidc_config_to_properties * Update rust/operator-binary/src/security/oidc.rs Co-authored-by: Techassi <git@techassi.dev> * Rename NoOidcTlsVerificationNotSupported -> SkippingTlsVerificationNotSupported --------- Co-authored-by: Techassi <git@techassi.dev>
1 parent 5f9de58 commit c851f51

File tree

8 files changed

+498
-97
lines changed

8 files changed

+498
-97
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ All notable changes to this project will be documented in this file.
3030
- Switch from `flow.xml.gz` to `flow.json.gz` to allow seamless upgrades to version 2.0 ([#675]).
3131
- Failing to parse one `NifiCluster`/`AuthenticationClass` should no longer cause the whole operator to stop functioning ([#662]).
3232
- NiFi will now use the JDK trust store when an OIDC provider uses WebPKI as CA ([#686], [#698]).
33+
- Fix OIDC endpoint construction in case the `rootPath` does not have a trailing slash ([#718]).
3334

3435
### Removed
3536

@@ -50,6 +51,7 @@ All notable changes to this project will be documented in this file.
5051
[#698]: https://github.com/stackabletech/nifi-operator/pull/698
5152
[#702]: https://github.com/stackabletech/nifi-operator/pull/702
5253
[#708]: https://github.com/stackabletech/nifi-operator/pull/708
54+
[#718]: https://github.com/stackabletech/nifi-operator/pull/718
5355

5456
## [24.7.0] - 2024-07-24
5557

Cargo.lock

Lines changed: 91 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)