Skip to content

docs(security): document STDIO transport security model#119

Merged
epugh merged 1 commit into
apache:mainfrom
adityamparikh:docs/security-stdio-mode
May 8, 2026
Merged

docs(security): document STDIO transport security model#119
epugh merged 1 commit into
apache:mainfrom
adityamparikh:docs/security-stdio-mode

Conversation

@adityamparikh
Copy link
Copy Markdown
Contributor

Summary

  • Adds docs/security/stdio.md documenting the security posture of the Solr MCP server when run in STDIO mode (the default).
  • Captures why STDIO has no in-process auth layer — and that this is intentional and spec-aligned — with citations to the MCP specification (Authorization, Transports, Security Best Practices), Spring AI MCP Security reference, spring-ai-community/mcp-security, and Spring Security/Boot Javadoc.
  • Pure docs change. No code, config, or behavior changes.

The TL;DR for reviewers: the MCP Authorization spec explicitly says STDIO implementations SHOULD NOT follow OAuth and should retrieve credentials from the environment instead. The doc records that fact and the operational guardrails (don't run elevated, treat SOLR_URL as deployer config, never write to stdout) for future contributors.

A follow-up doc for HTTP-mode security is referenced as planned but not included here.

Test plan

  • docs/security/stdio.md renders correctly on GitHub (table, blockquotes, links).
  • All external links resolve (MCP spec pages, Spring AI / Spring blog, Spring Security/Boot Javadoc).
  • No code paths touched; existing CI (unit + integration) is unaffected.

🤖 Generated with Claude Code

Adds docs/security/stdio.md capturing why STDIO mode intentionally has
no in-process auth layer, with operational guidance and citations to the
MCP specification (Authorization, Transports, Security Best Practices),
Spring AI MCP Security docs, spring-ai-community/mcp-security, and
relevant Spring Security/Boot references.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
@epugh epugh merged commit 38fca3d into apache:main May 8, 2026
4 checks passed
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