docs: Edge Proxy operational guide#7397
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Adds identity overrides, troubleshooting, production deployment, and architecture/scaling sections to the Edge Proxy doc, addressing the recurring support patterns catalogued in #7207.
6e990ec to
4c3eb39
Compare
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
Holmus
left a comment
There was a problem hiding this comment.
Really, really solid overall! most comments are nits about line breaks, eventually i stopped commenting on each one as it was just too many and i was not even sure whether they are mistakes or not.
Some real questions and comments but nearly there.
- Drop Edge enablement bullet from 403 troubleshooting - Drop empty client_side_key bullet from CI/CD section - Drop If-Modified-Since aside from architecture/scaling
Holmus
left a comment
There was a problem hiding this comment.
Awesome, thanks for this!
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Contributes to #7207
Extends
docs/docs/deployment-self-hosting/edge-proxy.mdwith operational guidance that the existing setup-and-config doc didn't cover, addressing the recurring support patterns catalogued in the issue:Use identity overrides in local evaluationrequirement, and the v2.21.1 deletion-propagation note.config.jsonas a secret in CI/CD.The Rust Edge Proxy section called out in the issue is intentionally out of scope for this PR.
How did you test this code?
Docs-only change, no runtime behaviour to test