Skip to content

add default connection profile and stdio rate limit#83

Merged
khelanmodi merged 5 commits into
mainfrom
xingfan/default-profile-rate-limit
May 27, 2026
Merged

add default connection profile and stdio rate limit#83
khelanmodi merged 5 commits into
mainfrom
xingfan/default-profile-rate-limit

Conversation

@xingfan-git
Copy link
Copy Markdown
Contributor

@xingfan-git xingfan-git commented May 21, 2026

Adds local stdio default profile support and related safety/configuration improvements.

  • Add DEFAULT_CONNECTION_PROFILE for trusted local stdio calls.
  • Rename local stdio trust config to TRUST_LOCAL_STDIO.
  • Require explicit profile authMode (connectionString or entra).
  • Add stdio rate limiting alongside HTTP/SSE rate limiting.
  • Update tests and stdio e2e scripts.

@khelanmodi khelanmodi merged commit df7a77d into main May 27, 2026
3 checks passed
xgerman pushed a commit to Azure/documentdb-agent-kit that referenced this pull request May 27, 2026
Upstream microsoft/documentdb-mcp#83 renamed ALLOW_UNAUTHENTICATED_STDIO
to TRUST_LOCAL_STDIO. The old name is no longer recognized by the server
at main; it has been silently ignored on every install since that PR
landed. Our installer kept working only because it also sets
AUTH_REQUIRED=false, which short-circuits the server's Entra startup
validator before TRUST_LOCAL_STDIO is ever consulted in the transport
switch. Rename everywhere so future readers don't see a dead env var:

- install.sh build_env_json (python + jq paths)
- install.ps1 Get-EnvJsonHashtable
- mcp.json
- gemini-extension.json (Gemini extension manifest)
- skills/mcp-setup/SKILL.md (Step 3, config template, troubleshooting)
- README.md (troubleshooting table)
- CHANGELOG.md (new entry + footnote on the 2026-05-21 entry)

Comments in install.sh / install.ps1 / SKILL.md keep an explicit
'previously ALLOW_UNAUTHENTICATED_STDIO' note pointing at the upstream
PR so users on older server builds know what to use instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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