[Docs] Add missing MCP per-user token env vars to config_settings#25471
[Docs] Add missing MCP per-user token env vars to config_settings#25471yuneng-berri merged 1 commit intomainfrom
Conversation
MCP_PER_USER_TOKEN_DEFAULT_TTL and MCP_PER_USER_TOKEN_EXPIRY_BUFFER_SECONDS were added in #25441 but not documented, causing test_env_keys.py to fail.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis PR adds two missing environment variables ( Confidence Score: 5/5Safe to merge — documentation-only fix with accurate defaults, unblocks CI on dependent PRs. The change is two documentation lines with values verified against the source constants. No logic, security, or schema concerns. No files require special attention.
|
| Filename | Overview |
|---|---|
| docs/my-website/docs/proxy/config_settings.md | Added two MCP per-user token env var entries with correct defaults, grouped logically with existing MCP OAuth settings. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[MCP Per-User OAuth Token Request] --> B{expires_in present?}
B -- Yes --> C[TTL = expires_in - MCP_PER_USER_TOKEN_EXPIRY_BUFFER_SECONDS\ndefault buffer: 60s]
B -- No --> D[TTL = MCP_PER_USER_TOKEN_DEFAULT_TTL\ndefault: 43200s / 12 hours]
C --> E[Store token in Redis with computed TTL]
D --> E
Reviews (1): Last reviewed commit: "[Docs] Add missing MCP per-user token en..." | Re-trigger Greptile
Summary
MCP_PER_USER_TOKEN_DEFAULT_TTLandMCP_PER_USER_TOKEN_EXPIRY_BUFFER_SECONDSwere introduced in #25441 but not added to the environment variables reference doc, causingtest_env_keys.pyto fail on all PRs branched from main after that merge.Changes
Added both env vars to the
environment variables - Referencetable indocs/my-website/docs/proxy/config_settings.md, grouped with the existing MCP OAuth entries.Testing
Ran
poetry run python ./tests/documentation_tests/test_env_keys.pylocally — passes.Type
📖 Documentation