Skip to content

Forward /.well-known/ to backend in custom Nginx configuration - #1107

Open
Harry-Chen wants to merge 1 commit into
rybbit-io:masterfrom
Harry-Chen:patch-1
Open

Forward /.well-known/ to backend in custom Nginx configuration#1107
Harry-Chen wants to merge 1 commit into
rybbit-io:masterfrom
Harry-Chen:patch-1

Conversation

@Harry-Chen

@Harry-Chen Harry-Chen commented Aug 8, 2026

Copy link
Copy Markdown

MCP client depends on this path for OAuth auto-discovery.

Summary by CodeRabbit

  • Documentation
    • Updated the custom Nginx self-hosting guide to proxy API and well-known paths correctly.
    • Clarified backend and client service configuration comments.
    • Fixed formatting in the log command example.

Copilot AI lite review requested due to automatic review settings August 8, 2026 06:07
@vercel

vercel Bot commented Aug 8, 2026

Copy link
Copy Markdown

@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.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6b745787-07a0-41e9-8f97-36b03728ba9f

📥 Commits

Reviewing files that changed from the base of the PR and between 9597d06 and b1cb228.

📒 Files selected for processing (1)
  • docs/content/docs/(docs)/self-hosting-guides/custom-nginx.mdx

📝 Walkthrough

Walkthrough

The self-hosting Nginx guide now proxies /api/ and /.well-known/ paths to the backend. It also updates a service comment and closes the Rybbit logs code block.

Changes

Self-hosting Nginx guide

Layer / File(s) Summary
Nginx routing and example corrections
docs/content/docs/(docs)/self-hosting-guides/custom-nginx.mdx
The backend proxy matches /api/ and /.well-known/. The client proxy comment uses “Client service.” The Rybbit logs example includes its closing Markdown fence.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: forwarding /.well-known/ requests to the backend in the custom Nginx configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the self-hosting “custom Nginx” documentation to route /.well-known/* requests to the backend, enabling OAuth auto-discovery for the MCP client.

Changes:

  • Expands backend proxy routing to include /.well-known/* alongside /api/* via a single location rule.
  • Renames location comments (“API requests” → “Backend service”, “Client app” → “Client service”).
  • Fixes the trailing code fence alignment at the end of the document.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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