Skip to content

secure mcp protocol - #1309

Closed
wenhuizhang wants to merge 3 commits into
modelcontextprotocol:mainfrom
AgentsSec:sec
Closed

secure mcp protocol#1309
wenhuizhang wants to merge 3 commits into
modelcontextprotocol:mainfrom
AgentsSec:sec

Conversation

@wenhuizhang

@wenhuizhangwenhuizhang commented Aug 26, 2025•
edited
Loading

Copy link
Copy Markdown

Security Features:

  • Secure annotations (src/mcp/server/fastmcp/secure/)
  • Transport security (src/mcp/server/transport_security.py)
  • Authentication middleware and token verification
  • Input/output validation and structured schemas

@wenhuizhang
wenhuizhang requested review from a team and ochafikAugust 26, 2025 21:50
@wenhuizhangwenhuizhang mentioned this pull request Aug 25, 2025
Closed
@wenhuizhangwenhuizhang changed the title Secsecure mcp protocolAug 26, 2025
@dsp-ant

dsp-ant commented Sep 2, 2025

Copy link
Copy Markdown
Member

Sorry, this PR seems to introduce a large set of features that are not specified in the protocol. The PR comes without any description whatsoever as to what it tries to achieve, how it goes about it and why it should be considered.

Please consider writing a SEP first https://modelcontextprotocol.io/community/sep-guidelines

@dsp-antdsp-ant closed this Sep 2, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ochafikochafikAwaiting requested review from ochafik

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wenhuizhang@dsp-ant