Docs: add practical server hardening checklist to SBP#2223
Docs: add practical server hardening checklist to SBP#2223TheodorNEngoy wants to merge 4 commits intomodelcontextprotocol:mainfrom
Conversation
|
Thanks for the contribution, @TheodorNEngoy. This likely should not be a separate doc. Take a look here: #2196 If there is anything missing from the SBP doc, we should add it there. |
|
Thanks for the pointer, makes sense. I just pushed an update that removes the standalone If you’d prefer this live elsewhere (or align wording/structure with #2196), I’m happy to adjust. |
|
Yeah, no need to modify it across two spec versions. I recommend keeping this open for now, and once #2196 merges, then just amend it with your suggestions and we can review them then. Helps avoid churn. Also, @TheodorNEngoy - I'd recommend structuring it in a way that is similar to what SBP already does. That is - instead of having a "hardening" checklist, structured it through the threat/mitigation/mistakes frame that you already might see in the doc. |
|
Makes sense. I pushed a small follow-up to reduce churn: this PR now only touches I’ll keep this open but pause further edits until #2196 lands; then I can rebase/amend and re-structure the additions in the same threat/mitigation/mistakes style as SBP (vs a standalone “hardening checklist”). |
Adds a short, implementation-agnostic server hardening checklist to the Security Best Practices doc (for common real-world MCP server footguns, especially when exposed over HTTP/SSE/WebSocket).
Covers:
Updated in both:
docs/specification/draft/basic/security_best_practices.mdxdocs/specification/2025-11-25/basic/security_best_practices.mdxDocs only; no behavior changes.