Skip to content

docs: drop the removed MCP query-token auth guidance - #238

Merged
buddies2705 merged 1 commit into
mainfrom
claude/unruffled-sutherland-f01878
Aug 7, 2026
Merged

docs: drop the removed MCP query-token auth guidance#238
buddies2705 merged 1 commit into
mainfrom
claude/unruffled-sutherland-f01878

Conversation

@buddies2705

@buddies2705 buddies2705 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

What

Fixes the broken anchor npm run build reports:

Broken anchor on source page path = /docs/mcp/build-a-trading-agent/:
 -> linking to /docs/mcp/mcp-server/#authentication-via-query-token-less-secure

Why not just re-anchor it

Commit 2bfb9b3 removed the "Authentication via query token (less secure)" section from docs/mcp/mcp-server.mdx as an incorrect auth method. The sentence holding the dangling link didn't just link to that section — it repeated its instruction:

For headless servers, append ?token=YOUR_TOKEN

Re-pointing the link would have left users following a method that was deliberately deleted. Both the link and the instruction are gone; the text now describes the OAuth 2.1 flow mcp-server.mdx actually documents, and links to the #first-connection-and-permissions section.

Confirmed with the docs owner: removing query-token auth from the docs is intended. OAuth 2.1 is the supported path. This PR finishes that removal rather than reviving it.

Second occurrence, not build-visible

docs/mcp/cursor.mdx:48 carried the same claim in an FAQ answer:

Can I use an API token instead of OAuth? → "Yes for headless setups — append ?token=YOUR_TOKEN to the MCP URL."

It has no link, so the build never flagged it, but it is the same instruction that was removed from the overview page. Fixed here so the method is gone from the MCP docs entirely rather than surviving in the one place the link checker can't see.

?token=YOUR_TOKEN elsewhere in the docs refers to streaming.bitquery.io WebSockets, which does support it — untouched. That's also the likely origin of the error: the deleted section explicitly reasoned by analogy from the WebSocket behaviour.

Side note for whoever runs the MCP server (not a docs issue)

The server's RFC 9728 metadata still advertises query-parameter bearer auth, which is now inconsistent with the documented position:

$ curl -s https://mcp.bitquery.io/.well-known/oauth-protected-resource
{"resource":"https://mcp.bitquery.io","authorization_servers":["https://account.bitquery.io"],
 "bearer_methods_supported":["header","query"], ...}

Flagging only so it isn't a surprise later — a client that reads this metadata may try query auth. Nothing in this PR depends on it, and it needs no action here.

Verification

npm run build passes; the broken-anchor warning is gone, and grepping the full log for broken|anchor returns nothing. Confirmed in the built HTML that id="first-connection-and-permissions" exists on the mcp-server page and that no token=YOUR_TOKEN survives under build/docs/mcp/.

🤖 Generated with Claude Code

Commit 2bfb9b3 removed the "Authentication via query token (less
secure)" section from docs/mcp/mcp-server.mdx as an incorrect auth
method, but left two inbound references to it in place.

build-a-trading-agent.mdx still linked to the deleted anchor, which
`npm run build` flagged as a broken anchor. Re-anchoring alone would
have been wrong: the surrounding sentence told users to append
`?token=YOUR_TOKEN` for headless servers, i.e. it repeated the same
method that was just removed as incorrect. Both now describe the
OAuth 2.1 flow the server actually documents, and the link points at
the "First Connection and Permissions" section.

cursor.mdx had no link, so the build never flagged it, but its FAQ
asserted the same `?token=YOUR_TOKEN` method. Fixed alongside so the
incorrect instruction is gone from the MCP docs entirely.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@buddies2705
buddies2705 merged commit 17a9c87 into main Aug 7, 2026
1 check 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.

1 participant