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

bug: URI parsing in the admin console results incorrect redirectUris and postLogoutRedirectUris stored #5748

Open
2 of 4 tasks
Fierce-Cat opened this issue Apr 18, 2024 · 1 comment
Assignees
Labels
bug Something isn't working status/planned We decided to do it.

Comments

@Fierce-Cat
Copy link

Describe the bug

In the application setting, input fields of Redirect URIs and Post Sign-out Redirect URIs will automatically parse URIs with non-ASCII characters, which will cause the client to fail to pass verification when sending requests to OIDC endpoint with its original redirect URI.

Expected behavior

Redirect URIs and Post Sign-out Redirect URIs should preserve the original input URI(s).
msedge_EJxfQEm99D

How to reproduce?

  • Enter URIs with non-English characters in Redirect URIs and Post Sign-out Redirect URIs
  • click save
  • URIs stored as parsed.

Context

  • Logto Cloud
  • Self-hosted, Logto version = 1.15.0
    • Container (Docker image)
    • Raw Node.js

Temporary solution

Use the Management API to change URIs above.

Screenshots

Current - after click save:
msedge_h4B69aULwd

Request from client failed:
image

@Fierce-Cat Fierce-Cat added the bug Something isn't working label Apr 18, 2024
@charIeszhao
Copy link
Member

Yeah, this is an acknowledged issue and we're trying to fix that in the next release. Thanks for reporting.

@charIeszhao charIeszhao self-assigned this Apr 22, 2024
@charIeszhao charIeszhao added the status/planned We decided to do it. label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status/planned We decided to do it.
Development

No branches or pull requests

2 participants