feat: enhance MCP authentication and allow configurable /mcp path #3061
+651
−804
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 在BasicHttpAuthenticatorHandler中添加Subject到Channel AttributeKey的存储 - 修复MCP工具调用时"try to use 'auth' command to authenticates"错误
- 配置文件中默认指定 mcp server path 为 /mcp - 如果配置文件中没有指定 mcp path 默认不开启 mcp server
- 创建 AbstractArthasTool 抽象基类，统一管理上下文和执行逻辑 - 统一同步和流式工具的执行模式 - 简化参数处理，移除冗余的参数校验
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
主要改动
实现 MCP 模块完整认证传递链路
BasicHttpAuthenticatorHandler中新增Subject到Channel AttributeKey的存储。"try to use 'auth' command to authenticates"错误。新增配置项：支持自定义 MCP path
mcp server path（默认值为/mcp）。mcp path，默认不开启 MCP server。