Skip to content

Validate custom agent MCP and HTTP hook URLs - #347180

Draft
Martin Aeschlimann (aeschli) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-custom-agent-editor-urls
Draft

Validate custom agent MCP and HTTP hook URLs#347180
Martin Aeschlimann (aeschli) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-custom-agent-editor-urls

Conversation

Copilot AI commented Nov 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Custom agent diagnostics missed malformed remote MCP and HTTP hook URLs, including https:// values without a host.

  • Validation: Require both URL types to parse as HTTP or HTTPS and include a hostname.
  • Example: These values now produce diagnostics: 
    mcp-servers:
  remote:
    type: http
    url: "https://"
    tools: [fetch]
hooks:
  preToolUse:
    - type: http
      url: "not-a-url"

Copilot AI balanced review requested due to automatic review settings November 9, 2026 21:21
Copilot AI reviewed Nov 9, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

Copilot AI linked an issue Nov 9, 2026 that may be closed by this pull request
Custom agent editor accepts malformed MCP server and HTTP hook URLs #344381
Open
@aeschli
Validate custom agent HTTP URLs
c1fe2a5 
Co-authored-by: aeschli <6461412+aeschli@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix malformed MCP server and HTTP hook URLs in custom agent editor Validate custom agent MCP and HTTP hook URLs Nov 9, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@aeschli Martin Aeschlimann (aeschli) Awaiting requested review from aeschli

At least 1 approving review is required to merge this pull request.

Assignees

@aeschli Martin Aeschlimann (aeschli)

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Custom agent editor accepts malformed MCP server and HTTP hook URLs

3 participants

@aeschli