Skip to content

Conversation

@eleftherias
Copy link
Member

Description

Document --remote-resource-indicator flag which can be used when running remote MCP servers.

Related issues/PRs

stacklok/toolhive#2497

Screenshots

Merge checklist

Content

  • New pages include a frontmatter section with title and description at a minimum
  • Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files
  • Redirects added to vercel.json for moved, renamed, or deleted pages (i.e., if the URL slug changed)

Reviews

  • Content has been reviewed for technical accuracy
  • Content has been reviewed for spelling, grammar, and style

Copilot AI review requested due to automatic review settings November 10, 2025 08:53
@vercel
Copy link

vercel bot commented Nov 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-website Ready Ready Preview Comment Nov 10, 2025 8:59am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for the resource indicator feature (RFC 8707) when authenticating to remote MCP servers. The new section explains how ToolHive handles resource indicators for OAuth token requests.

  • Adds a new section explaining RFC 8707 resource indicator support
  • Documents default behavior (automatic use of server URL as resource indicator)
  • Provides example of overriding the resource indicator with --remote-auth-resource flag

Comment on lines +746 to +748
thv run https://api.example.com/mcp \
... \
--remote-auth-resource https://api.example.com
Copy link

Copilot AI Nov 10, 2025

Choose a reason for hiding this comment

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

The ellipsis (...) placeholder in the command example should be replaced with actual flag examples or removed. Per the writing style guide, examples should be practical and real-world. Consider showing a complete example with the other authentication flags mentioned earlier in the document (like --remote-auth-issuer, --remote-auth-client-id).

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

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

Ignoring this suggestion, since the current format aligns with the examples for custom authentication timeout, skip browser auth, and custom scopes

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@eleftherias eleftherias merged commit 7e34960 into main Nov 10, 2025
8 checks passed
@eleftherias eleftherias deleted the resource-indicator branch November 10, 2025 15:55
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.

3 participants