Modify custom Nginx config for OIDC Discovery support - #1126
Conversation
Updated Nginx configuration to include OIDC Discovery path.
|
@Harry-Chen is attempting to deploy a commit to the goldflag's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe custom Nginx guide documents OIDC Discovery support and routes both ChangesCustom Nginx OIDC routing
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a localized documentation update for custom Nginx OIDC Discovery support, with no actionable merge-blocking risk remaining beyond normal checks and review. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the self-hosting “custom Nginx” guide to ensure OAuth/OIDC discovery endpoints under /.well-known/* are proxied to the backend (matching the MCP client’s auto-discovery expectations), including the custom-port configuration example.
Changes:
- Documented that
/.well-known/*should be forwarded to the backend for OIDC discovery. - Updated the custom-ports Nginx snippet to proxy both
/api/*and/.well-known/*to the backend.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
#1107 has added some but missed another nginx configuration item.
Summary by CodeRabbit
.well-knownroute supports OIDC Discovery..well-knownrequests to the backend.