SEP-2678: Introduce additional error codes to protocol#2678
Open
MatthewKhouzam wants to merge 1 commit into
Open
SEP-2678: Introduce additional error codes to protocol#2678MatthewKhouzam wants to merge 1 commit into
MatthewKhouzam wants to merge 1 commit into
Conversation
090e3cc to
cca3bf9
Compare
Introduce -32000/-32001/-32002 to match the implementation in FastMCP. This change paves the way towards "interceptors" or "middleware" standardization. This change was assisted with claude opus 4.7 Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
cca3bf9 to
dd6cb6c
Compare
Author
|
@reviewers. I would like to talk about if my approach is correct and inline with your roadmap before merging. This patch is trivial, but I don't know if it's useful for your goals. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Introduce -32000/-32001/-32002 to match the implementation in FastMCP.
This change paves the way towards "interceptors" or "middleware" standardization.
Motivation and Context
The end goal is to better secure MCP against several attacks. FastMCP's middleware proxy design looks like a great solution to add local verification. This change would bring the error codes from that design pattern into the proper MCP. This should have 0 breaking changes as the error codes are already in the range of -32000 - -32099. This would just bring transparency to the error codes.
How Has This Been Tested?
This is a "documentation" and protocol change adding error messages to match those of FastMCP's implementation.
Breaking Changes
Types of changes
Checklist
Additional context
This code was generated with the assistance of claude opus 4.7 (the SEP was largely assisted for linguistic reasons). It was verified by a person.