docs: drop the removed MCP query-token auth guidance - #238
Merged
Conversation
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Fixes the broken anchor
npm run buildreports:Why not just re-anchor it
Commit 2bfb9b3 removed the "Authentication via query token (less secure)" section from
docs/mcp/mcp-server.mdxas an incorrect auth method. The sentence holding the dangling link didn't just link to that section — it repeated its instruction: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.mdxactually documents, and links to the#first-connection-and-permissionssection.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:48carried the same claim in an FAQ answer: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_TOKENelsewhere in the docs refers tostreaming.bitquery.ioWebSockets, 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:
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 buildpasses; the broken-anchor warning is gone, and grepping the full log forbroken|anchorreturns nothing. Confirmed in the built HTML thatid="first-connection-and-permissions"exists on the mcp-server page and that notoken=YOUR_TOKENsurvives underbuild/docs/mcp/.🤖 Generated with Claude Code