Skip to content

Conversation

wyangsun
Copy link

@wyangsun wyangsun commented Aug 6, 2025

  1. Add MCPURL for different hostname.
  2. Use MCPURL as MCP server Oauth URL.

@wyangsun wyangsun requested a review from Copilot August 7, 2025 09:57
Copilot

This comment was marked as off-topic.

Signed-off-by: Yang Wang <wyangsun@163.com>
Signed-off-by: Yang Wang <wyangsun@163.com>
Signed-off-by: Yang Wang <wyangsun@163.com>
Copy link

@tommynguyen-vungle tommynguyen-vungle left a comment

Choose a reason for hiding this comment

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

LGTM

@wyangsun wyangsun merged commit 3593191 into main Aug 8, 2025
4 of 5 checks passed
@wyangsun wyangsun deleted the PE-7208-support-hostname branch August 8, 2025 03:28
tommynguyen-vungle pushed a commit that referenced this pull request Aug 17, 2025
* PE-7046: Deploy Trino MCP Server

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* remove default annotations for ALB ingress controller

* PE-7208: Support hostname in MCP URL (#3)

* PE-7208: Support hostname in MCP URL

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>
tommynguyen-vungle added a commit that referenced this pull request Aug 17, 2025
* PE-7046: Deploy Trino MCP Server

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* remove default annotations for ALB ingress controller

* PE-7208: Support hostname in MCP URL (#3)

* PE-7208: Support hostname in MCP URL

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>
Co-authored-by: Yang Wang <wyangsun@163.com>
tommynguyen-vungle pushed a commit that referenced this pull request Aug 19, 2025
* PE-7208: Support hostname in MCP URL

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>
tommynguyen-vungle pushed a commit that referenced this pull request Aug 19, 2025
* PE-7046: Deploy Trino MCP Server

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* remove default annotations for ALB ingress controller

* PE-7208: Support hostname in MCP URL (#3)

* PE-7208: Support hostname in MCP URL

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>
tommynguyen-vungle added a commit that referenced this pull request Aug 19, 2025
* PE-7046: Deploy Trino MCP Server

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* PE-7046: Fix workflow

* remove default annotations for ALB ingress controller

* PE-7208: Support hostname in MCP URL (#3)

* PE-7208: Support hostname in MCP URL

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

* Update oauth.md

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>

---------

Signed-off-by: Yang Wang <wyangsun@163.com>
Co-authored-by: Yang Wang <wyangsun@163.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.

2 participants