Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix Edge Case Handling for Zapier MCP Servers#311

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
prasys wants to merge3 commits intomcp-use:main
base:main
Choose a base branch
Loading
fromprasys:zapier-intergration

Conversation

@prasys
Copy link

@prasysprasys commentedOct 16, 2025
edited
Loading

Pull Request Description

Changes

Added special handling for Zapier MCP server endpoints to differentiate between OAuth-authenticated (/a/) and server-specific URL (/s/) connection types.

(Types of connectivity with Zapier)
image

The OAuth discovery process now intelligently skips discovery for server-specific Zapier MCP server URLs while maintaining full OAuth flow support.

Implementation Details

  1. Changes done on_discover_metadata method inOAuth class (libraries/python/mcp_use/auth/oauth.py)
    • Added URL parsing logic to detect Zapier MCP server endpoints
    • https://mcp.zapier.com/api/mcp/s/ endpoint (Pre-authenticated)**: RaisesOAuthDiscoveryError so that it will be handled as no authentication
    • https://mcp.zapier.com/api/mcp/a/* endpoint (OAuth)**: Proceeds with standard OAuth metadata discovery flow
    • Maintains existing edge case pattern (similar to GitHub API handling)

Example Usage (Before)

# Before: OAuth discovery would fail for Zapier /s/ endpoints with generic errorfrommcp_useimportMCPClient,MCPAgent# Zapierconfig= {"mcpServers": {"zapier": {"url":"https://mcp.zapier.com/api/mcp/s/XXXXXXXXX/mcp",        }    }}

This would attempt OAuth discovery

Example Usage (After)

No change, the system will handle as if there is no authentication

Testing

Describe how you tested these changes:

I've performed manual testing on both the oauth and direct url with Zapier

Backwards Compatibility

N/A - it does not impact any existing functionality as it is mainly to handle Zapier edge case scenario

Related Issues

Closes#309

@Pederzh
Copy link
Contributor

@claude

claude[bot] reacted with eyes emoji

@claude
Copy link
Contributor

claudebot commentedOct 16, 2025
edited
Loading

Claude encountered an error ——View job

Failed with exit code 128

I'll analyze this and get back to you.

@mcp-usemcp-use deleted a comment fromclaudebotNov 15, 2025
@mcp-usemcp-use deleted a comment fromclaudebotNov 15, 2025
@mcp-usemcp-use deleted a comment fromclaudebotNov 15, 2025
@tonxxd
Copy link
Contributor

@claude

claude[bot] reacted with eyes emoji

@claude
Copy link
Contributor

claudebot commentedNov 15, 2025
edited
Loading

Claude encountered an error ——View job

Failed with exit code 128

I'll analyze this and get back to you.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Zapier MCP Server “server-specific URL” triggers unexpected OAuth discovery

4 participants

@prasys@Pederzh@tonxxd@pietrozullo

[8]ページ先頭

©2009-2025 Movatter.jp