Skip to content

Conversation

tommynguyen-vungle
Copy link

@tommynguyen-vungle tommynguyen-vungle commented Aug 20, 2025

Don't dictate, let client decide redirect URI

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Copy link

@wyangsun wyangsun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tommynguyen-vungle tommynguyen-vungle merged commit 2673e1a into main Aug 20, 2025
6 checks passed
@tommynguyen-vungle tommynguyen-vungle deleted the PE-6985-fix-trino-oauth-2 branch August 20, 2025 03:10
tommynguyen-vungle added a commit that referenced this pull request Aug 20, 2025
* feat(oauth): add redirect URIs for mcp-remote compatibility (#5)

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>

* feat(oauth): allow client registration of redirect URIs (#6)

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>

---------

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>
tommynguyen-vungle added a commit that referenced this pull request Aug 20, 2025
* feat(oauth): add redirect URIs for mcp-remote compatibility (#5)

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>

* feat(oauth): allow client registration of redirect URIs (#6)

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>

---------

Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
Co-authored-by: Tommy Nguyen <tuannvm@hotmail.com>
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.

3 participants