Skip to content

Enforce HTTP MCP limit caps - #3264

Merged
Widthdom merged 2 commits into
mainfrom
codex/fix-issue3157-3227
Jun 4, 2026
Merged

Enforce HTTP MCP limit caps#3264
Widthdom merged 2 commits into
mainfrom
codex/fix-issue3157-3227

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Enforce hard maximums for CDIDX_MCP_HTTP_MAX_REQUEST_BYTES and CDIDX_MCP_HTTP_MAX_QUEUE_DEPTH before the HTTP listener starts.
  • Return CLI usage errors for oversized HTTP MCP limit configuration and document accepted ranges.
  • Add HTTP transport and CLI regression coverage, including positive integer overflow environment values.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~HttpMcpTransportTests" -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~ProgramCliTests.Mcp_HttpOversizedLimitEnvironmentReturnsUsageError" -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll . --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Adversarial review: no blocking/actionable issues found.

Full dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false --no-build passed on the immediately preceding latest-base run; after origin/main advanced again, the branch was cherry-picked onto the current origin/main and the targeted tests/build/checks above were rerun.

Changelog

  • changelog.d/unreleased/3157-3227.fixed.md

Fixes #3157
Fixes #3227

@Widthdom
Widthdom force-pushed the codex/fix-issue3157-3227 branch from 74e2966 to 69dfa4a Compare June 4, 2026 15:41
@Widthdom
Widthdom merged commit dd38d2b into main Jun 4, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue3157-3227 branch June 4, 2026 16:55
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.

HTTP MCP request body and queue limit env vars should enforce upper bounds HTTP MCP request limit environment variables should have hard maximums

1 participant