-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Open
Labels
authRelated to authentication (Bearer, JWT, OAuth, WorkOS) for client or server.Related to authentication (Bearer, JWT, OAuth, WorkOS) for client or server.enhancementImprovement to existing functionality. For issues and smaller PR improvements.Improvement to existing functionality. For issues and smaller PR improvements.
Description
Enhancement
Azure doesn't support the resource parameter in OAuth authorize requests, so the AzureProvider removes it.
We've run into the same issue when using Azure with Auth0. Could we make a similar change to the Auth0Provider, or maybe add a forward_resource: bool = true argument to the OAuthProxy class, similar to the forward_pkce argument, to allow servers to remove the resource parameter?
We're currently monkey patching Auth0Provider to remove the resource parameter similar to how it's done in the AzureProvider, so we have a workaround, but I feel the configuration option would be cleaner.
SHINMH
Metadata
Metadata
Assignees
Labels
authRelated to authentication (Bearer, JWT, OAuth, WorkOS) for client or server.Related to authentication (Bearer, JWT, OAuth, WorkOS) for client or server.enhancementImprovement to existing functionality. For issues and smaller PR improvements.Improvement to existing functionality. For issues and smaller PR improvements.