Skip to content

FR-6380 Point the MCP endpoint at mcp.fortrabbit.com/mcp - #4

Merged
pheeque1 merged 2 commits into
mainfrom
fr-6380-append-the-mcp-path-to-the-endpoint
Sep 1, 2026
Merged

FR-6380 Point the MCP endpoint at mcp.fortrabbit.com/mcp#4
pheeque1 merged 2 commits into
mainfrom
fr-6380-append-the-mcp-path-to-the-endpoint

Conversation

@frank-laemmer

Copy link
Copy Markdown
Member

ServerURL moves from https://api.fortrabbit.com/mcp to https://mcp.fortrabbit.com/mcp.

The old endpoint is gone — POST https://api.fortrabbit.com/mcp now answers 403 with an empty body and no WWW-Authenticate header. The new host serves JSON-RPC on the /mcp path only; the bare host answers 405 Method Not Allowed (Allow: GET).

Two commits: the first moves to the new host, the second adds the path that the host actually serves.

Because service.go:202 compares a registration against ServerURL, any client already pointing at the old value now reads as stale, so frbit mcp install repairs it rather than leaving a dead entry behind.

Matching changes are already released elsewhere:

  • fortrabbit/agent-skills — pushed to main (a32a24e)
  • fortrabbit/frbit-frontend — released as v20260831, live on www, docs and the dashboard

go build ./... passes; internal/agentmcp and internal/cmd/mcp tests pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ArDJv66awmc1d9RMHUYBrx

frank-laemmer and others added 2 commits August 28, 2026 12:51
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013G3f9pdDE232yz1pA78eqS
The bare host answers only GET and 404s; JSON-RPC is served at
https://mcp.fortrabbit.com/mcp. Existing registrations now compare
as stale, so `frbit mcp install` repairs them.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ArDJv66awmc1d9RMHUYBrx
@frank-laemmer
frank-laemmer requested a review from pheeque1 August 31, 2026 18:31

@pheeque1 pheeque1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks.

@pheeque1
pheeque1 merged commit 4a022fb into main Sep 1, 2026
2 checks passed
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