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

Azure AD OAUTH requires fully qualified scope (https://graph.microsoft.com/offline_access) #1845

Closed
GhimBoon opened this issue Feb 3, 2025 · 4 comments
Labels
bug Something isn't working needs-triage

Comments

@GhimBoon
Copy link
Contributor

GhimBoon commented Feb 3, 2025

Describe the bug
AzureAD authentication started failing from 2.0.603 when offline_access was added without the Microsoft Graph prefix.

To Reproduce
Steps to reproduce the behavior:

  1. Upgrade to 2.0.603 or above
  2. Login using AzureAD
  3. Will encounter 502 error

Expected behavior
Login will work as expected

Screenshots
Forgot to take the screenshot

Additional context
Patched the scope to https://graph.microsoft.com/offline_access and it started working. Some production environments require fully qualified scopes to work.

@ajosegun
Copy link

ajosegun commented Feb 3, 2025

Image

Seems I have the same issue

@fcestari
Copy link

fcestari commented Feb 4, 2025

The PR #1846 actually had a different behavior to me - the refresh_token are no longer saved in the user metadata as per #1599

@GhimBoon
Copy link
Contributor Author

GhimBoon commented Feb 6, 2025

Update: I'm reverting it in #1869.

My issue is actually caused by kubernetes nginx that got a 502 due to the limited proxy_buffer sizing. The offline_access refresh_tokens requires a larger buffer size that kubernetes doesn't handle by default.

Adding these to my ingress solved it:
nginx.ingress.kubernetes.io/proxy-buffer-size: "64k"
nginx.ingress.kubernetes.io/proxy-buffers: "8 64k"

@GhimBoon
Copy link
Contributor Author

GhimBoon commented Feb 9, 2025

Closing this issue since this is not a bug.

@GhimBoon GhimBoon closed this as completed Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-triage
Projects
None yet
Development

No branches or pull requests

3 participants