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

Update redirect url definition for wildcard url allowed app types #4980

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mpmadhavig
Copy link
Contributor

Purpose

This PR contains the improvement for the below explanation.

The support for wild card in sub domains looks intentional to support use cases such as mobile apps requiring dynamic URL handling or tenanted URL handling.

However, its usage for general web-based redirect URIs may not align with OAuth 2.0 best practices.

When comparing client redirect URIs against pre-registered URIs, authorization servers MUST utilize exact string matching except for port numbers in localhost redirection URIs of native apps.

Regarding the customer recommendation,
AFAIK, Standard based templates used for clients (mobile or web-based) using standard protocols. But its recommended > to follow best practises,

  • For web-based applications, customers should use exact URLs or implement logic to dynamically register specific redirect URIs as needed.

  • For mobile apps leveraging deep links, wildcard support may be acceptable, but it must be implemented securely and limited in scope.

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