Skip to content

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

Merged
merged 7 commits into from
Jul 3, 2025
Merged

Conversation

vystrcild
Copy link
Contributor

No description provided.

@vystrcild vystrcild requested a review from MQ37 July 1, 2025 08:16
@vystrcild vystrcild requested a review from TC-MO as a code owner July 1, 2025 08:16
Copy link

cursor bot commented Jul 1, 2025

🚨 BugBot couldn't run

BugBot is experiencing high demand right now. Try again in a few minutes by commenting "bugbot run" (requestId: serverGenReqId_6e35a67b-2808-411d-b2cd-a17013f453b3).

@vystrcild vystrcild changed the title Oauth + adding multiple Actors for Streamable/SSE docs: Oauth + adding multiple Actors for Streamable/SSE Jul 1, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit c1439a8 and is ready at https://pr-1660.preview.docs.apify.com!

@@ -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)
Copy link
Contributor

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

Copy link
Contributor Author

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

Copy link
Contributor

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>
@apify-service-account
Copy link

Preview for this PR was built for commit d29169c and is ready at https://pr-1660.preview.docs.apify.com!

Copy link
Contributor

@TC-MO TC-MO left a 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

vystrcild and others added 2 commits July 1, 2025 13:53
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit f86e138 and is ready at https://pr-1660.preview.docs.apify.com!

1 similar comment
@apify-service-account
Copy link

Preview for this PR was built for commit f86e138 and is ready at https://pr-1660.preview.docs.apify.com!

@vystrcild vystrcild merged commit d681068 into master Jul 3, 2025
10 checks passed
@vystrcild vystrcild deleted the fix/mcp branch July 3, 2025 10:45
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.

4 participants