Skip to content

mcp: add case senstive tests to the parser#43463

Merged
botengyao merged 1 commit intoenvoyproxy:mainfrom
botengyao:add_case_sensitive_test
Feb 13, 2026
Merged

mcp: add case senstive tests to the parser#43463
botengyao merged 1 commit intoenvoyproxy:mainfrom
botengyao:add_case_sensitive_test

Conversation

@botengyao
Copy link
Member

@botengyao botengyao commented Feb 12, 2026

verify that a potential smuggling is not the problem.

{
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "name": "backend__greet",
    "Name": "backend__secretTool",  // ← different casing
    "arguments": {"name": "Exploit"}
  }

#39174

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Boteng Yao <boteng@google.com>
Copy link
Member

@agrawroh agrawroh left a comment

Choose a reason for hiding this comment

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

I was thinking if there are opportunities to parameterize the tests as they all seem to be validating the same thing. But maybe not as they are asserting on different sub-sections of the input JSON.

@botengyao botengyao merged commit 805a35e into envoyproxy:main Feb 13, 2026
28 checks passed
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