Skip to content

Conversation

@478320
Copy link
Contributor

@478320 478320 commented Nov 8, 2025

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

In this PR, I fixed a bug in the Mcp Server Plugin, a bug in the Mcp Client, and two issues in the implementation of mcp examples. The details are as follows:

  1. When registering a Selector with the path "/mcp/" in the Mcp Server Selector, the actual key created for the mcpServer is "/mcp/". However, when modifying the Rule to add a tool, the lookup path is "/mcp", which causes the tool to not be added correctly.
  2. The Mcp Client could not add the Context Path to the tool's URL.
  3. There was an issue with the case sensitivity of the HTTP GET method configuration in the Mcp example.
  4. The context path was modified in the Mcp example.

@478320 478320 changed the title Fix shenyu examples mcp Fix shenyu mcp bugs Nov 8, 2025
@Aias00 Aias00 merged commit ae6ddc5 into apache:master Nov 8, 2025
42 checks passed
@Aias00 Aias00 added this to the 2.7.0.3 milestone Nov 20, 2025
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