Skip to content

Commit 00d7fe3

Browse files
committed
fix: bound Entra group pagination, keep all memberships, add HTTP timeouts
Follow-ups to the @odata.nextLink pagination fix in #859: - Add a 60s request / 15s connect timeout to every user-info-fetcher outbound HTTP backend (Keycloak, Entra, XFSC AAS) via a shared utils::http::client_builder(); reqwest otherwise waits indefinitely. - Keep every directory membership by requesting /memberOf instead of restricting to microsoft.graph.group, so directory roles and administrative units are still returned. - Cap @odata.nextLink following at 100 pages as a backstop against a cyclic or runaway link, warning if the group list may be truncated. - Bump the yanked spin 0.9.8 to 0.9.9 (matching #857, which only landed on release-26.7) so cargo-deny passes, and regenerate Cargo.nix for the new wiremock dev-dependency.
1 parent 5970759 commit 00d7fe3

7 files changed

Lines changed: 313 additions & 20 deletions

File tree

CHANGELOG.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,14 @@ All notable changes to this project will be documented in this file.
2222
- BREAKING: Removed product-config machinery. This is a breaking change in terms of configuration.
2323
Users relying on the product-config `properties.yaml` file have to set these properties via the CRD.
2424
JsonPatches has been renamed to JsonPatch ([#842]).
25+
- Apply a 60s request and 15s connect timeout to all user-info-fetcher outbound HTTP backends
26+
(Keycloak, Entra, XFSC AAS), which previously waited indefinitely ([#859]).
27+
28+
### Fixed
29+
30+
- The Entra user-info-fetcher backend now follows Microsoft Graph's `@odata.nextLink` pagination,
31+
so users with more group memberships than fit on a single response page receive their complete
32+
group list. Following the link is bounded to 100 pages as a safeguard ([#859]).
2533

2634
### Removed
2735

@@ -38,6 +46,7 @@ All notable changes to this project will be documented in this file.
3846
[#842]: https://github.com/stackabletech/opa-operator/pull/842
3947
[#843]: https://github.com/stackabletech/opa-operator/pull/843
4048
[#851]: https://github.com/stackabletech/opa-operator/pull/851
49+
[#859]: https://github.com/stackabletech/opa-operator/pull/859
4150

4251
## [26.3.0] - 2026-03-16
4352

Cargo.lock

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

Cargo.nix

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

0 commit comments

Comments
 (0)