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

Incorrect documentation I think #1054

Open
5 of 6 tasks
mjaggard opened this issue Apr 18, 2024 · 0 comments
Open
5 of 6 tasks

Incorrect documentation I think #1054

mjaggard opened this issue Apr 18, 2024 · 0 comments
Labels

Comments

@mjaggard
Copy link

mjaggard commented Apr 18, 2024

Checklist:

  • I am using the latest release
  • I searched for existing GitHub issues
  • I read the documentation
  • I verified the client configuration matches the information in the identity provider (or I am using dynamic client registration)
  • I am either using a custom URI scheme or https with App Links for client redirect.
  • I can reproduce the issue in the demo app (optional)

Configuration

  • Version: N/A
  • Integration: ReactNative
  • Identity provider: Google Android

Issue Description

The documentation says

We recommend using a custom scheme based redirect URI (i.e. those of form my.scheme:/path), as this is the most widely supported across all versions of Android. To avoid conflicts with other apps, it is recommended to configure a distinct scheme using "reverse domain name notation". This can either match your service web domain (in reverse) e.g. com.example.service or your package name com.example.app or be something completely new as long as it's distinct enough. Using the package name of your app is quite common but it's not always possible if it contains illegal characters for URI schemes (like underscores) or if you already have another handler for that scheme - so just use something else.

However if you enter something other than your package ID, you cannot specify the redirect URL correctly when creating Google's Android-specific credentials.

@mjaggard mjaggard added the bug label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant