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

Backports to 0.7.x #294

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Backports to 0.7.x #294

merged 4 commits into from
Nov 25, 2024

Conversation

dmulder
Copy link
Collaborator

@dmulder dmulder commented Nov 15, 2024

Fixes #291
Fixes #290

Checklist

  • This pr contains no AI generated code
  • cargo fmt has been run
  • cargo clippy has been run
  • A functionality test has been added
  • make test has been run and passes

The graph url isn't always the default. US gov
tenants (amoung possibly others) utilize a
different graph url.

BUG: #291

Signed-off-by: David Mulder <[email protected]>
A 15 second timeout ensures banned users are
forgotten almost immediately, but it also
drammatically increases network traffic, slows
down Himmelblau, and causes AADSTS50196 errors on
some tenants ("The server terminated an operation
because it encountered a client request loop").
A 5 minute timeout is more reasonable, and
mirrors Winbind's behavior. This timeout can
be decreased via configuration if necessary.

BUG: #290

Signed-off-by: David Mulder <[email protected]>
Signed-off-by: David Mulder <[email protected]>
@dmulder dmulder changed the title Government tenant fixes backport to 0.7.x Backports to 0.7.x Nov 15, 2024
@dmulder dmulder force-pushed the stable-0.7.x_gov_tenant_fixes branch from 394e26a to 6c661d5 Compare November 15, 2024 18:03
This is to enable local group capabilities.

Signed-off-by: David Mulder <[email protected]>
@dmulder dmulder merged commit 8007bf3 into stable-0.7.x Nov 25, 2024
5 checks passed
@dmulder dmulder deleted the stable-0.7.x_gov_tenant_fixes branch November 25, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant