Skip to content

fix(cli): honor explicit URL transport - #5529

Open
mikamikasuki wants to merge 2 commits into
PrefectHQ:mainfrom
mikamikasuki:fix/fmc-0052-explicit-url-transport
Open

mikamikasuki wants to merge 2 commits into
PrefectHQ:mainfrom
mikamikasuki:fix/fmc-0052-explicit-url-transport

Conversation

@mikamikasuki

Copy link
Copy Markdown
Contributor

Description

The client CLI does not reliably honor an explicit --transport for URL targets. For example, an SSE URL with a query string can be rewritten by appending /sse to the query value, while an explicitly selected HTTP transport can still be inferred as SSE from the URL path.

Honor the selected transport and preserve URL query parameters when applying transport-specific path handling.

Closes #5525

Contribution type

  • Bug fix (simple, well-scoped fix for a clearly broken behavior)
  • Documentation improvement
  • Enhancement

Checklist

  • This PR addresses an existing issue (or fixes a self-evident bug)
  • I have read CONTRIBUTING.md
  • I have added tests that cover my changes
  • I have run uv run prek run --all-files and all checks pass
  • I have self-reviewed my changes
  • If I used an LLM, it followed the repo's contributing conventions (not generic output)

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

This pull request is awaiting maintainer assignment. Linked PRs stay open while maintainers triage the issue; this check remains failing until assignment.

Per CONTRIBUTING.md, an external PR must reference an issue that's assigned to its author. To get there:

  1. A maintainer will review the issue and decide whether to assign you. Please don't comment just to request assignment.

Once you're assigned and the link is present, this check is re-run automatically. Previously gate-closed PRs also reopen — no further action needed.

Maintainers: reopen this PR or remove the missing-issue-link label to bypass this check.

@marvin-context-protocol marvin-context-protocol Bot added bug Something isn't working. Reports of errors, unexpected behavior, or broken functionality. cli Related to FastMCP CLI commands (run, dev, install) or CLI functionality. labels Oct 6, 2026
🤖 Generated with Codex

This branch has not been deployed

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

Labels

bug Something isn't working. Reports of errors, unexpected behavior, or broken functionality. cli Related to FastMCP CLI commands (run, dev, install) or CLI functionality. missing-issue-link

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Client CLI --transport is not reliably honored for URL targets

1 participant