-
Notifications
You must be signed in to change notification settings - Fork 109
docs: Oauth + adding multiple Actors for Streamable/SSE #1660
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Changed order of ways how to connect - Added mention for oAuth
🚨 BugBot couldn't runBugBot is experiencing high demand right now. Try again in a few minutes by commenting "bugbot run" (requestId: serverGenReqId_6e35a67b-2808-411d-b2cd-a17013f453b3). |
Preview for this PR was built for commit |
@@ -34,6 +34,20 @@ Before you start, make sure you have the following: | |||
1. _Apify API Token:_ Get your personal API token from the **Integrations** section in [Apify Console](https://console.apify.com/account#/integrations). This token will be used to authorize the MCP server to run Actors on your behalf. | |||
1. _MCP client:_ An AI agent or client that supports MCP. This could be Anthropic Claude for Desktop, a VS Code extension with MCP support, Apify’s web-based Tester MCP Client, or any custom client implementation. See supported MCP clients in [official documentation](https://modelcontextprotocol.io/clients). | |||
|
|||
## Example usage (Streamable HTTP with Oauth) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kind of off topic but food for thought.
Wouldn't it make more sene to have:
Example usage
small intro paragraph
Streamable HTTP with OAuth
XYZ
Example usage (local stdio with Claude for Desktop)
XYZ
I can prepare separate PR with this approach, I think it makes it look cleaner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it still evolving and we are adding more stuff as we go. Maybe it makes sense to polish it little bit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once this is merged, I'll go over the whole doc and try to reformat it and clean it up
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Preview for this PR was built for commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just OAuth typo, otherwise LGTM
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Preview for this PR was built for commit |
1 similar comment
Preview for this PR was built for commit |
No description provided.