Skip to content

Fix 307 Temporary Redirect when use streamable_http #781

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

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open
Changes from 10 commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
0bf3521
Update streamable_http_path to /mcp/ to avoid 307 Temporary Redirect
chi2liu May 22, 2025
dd00227
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 23, 2025
d89e737
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
7d6e14f
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
a84399b
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
900c2dd
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 26, 2025
1a638ad
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
b3bf4c9
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
645e1ab
Streamable HTTP Trailing Slash Compatibility
chi2liu May 26, 2025
3e9e33f
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 27, 2025
7ef5f8d
Clean logic (#3)
vectorstain May 27, 2025
303cac8
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 27, 2025
0d05075
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 28, 2025
898fc88
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
45a5cd8
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
04f637c
Update test_streamable_http.py
chi2liu May 28, 2025
2377b70
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
09e403c
Merge branch 'fix-307-Temporary-Redirect' of https://github.com/chi2l…
May 28, 2025
a03c62e
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
10ccbbf
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
e102e7d
add unit test for Streamable HTTP Trailing Slash Compatibility
May 28, 2025
a72ffe8
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 29, 2025
6cf74f1
change Mount to Routers
May 29, 2025
2748a75
fix tests
May 29, 2025
b726c86
redirect_slashes to False
May 29, 2025
3b03b83
Update server.py
chi2liu May 29, 2025
d2ee4df
fix typo
May 29, 2025
355c882
Merge branch 'fix-307-Temporary-Redirect' of https://github.com/chi2l…
May 29, 2025
86da08d
fix typo
May 29, 2025
3d86e61
fix
May 29, 2025
cc9f3d9
Update server.py
chi2liu May 29, 2025
cce6d2f
Update test_server.py
chi2liu May 29, 2025
04c4bcf
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 29, 2025
1d9632b
Fix 307 temporary redirect refactor (#4)
vectorstain May 29, 2025
2761e8f
Merge branch 'main' into fix-307-Temporary-Redirect
chi2liu May 29, 2025
2aa0428
Update server.py
chi2liu May 29, 2025
5841a55
fix format
May 30, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 25 additions & 0 deletions src/mcp/server/fastmcp/server.py
Original file line number Diff line number Diff line change
Expand Up @@ -830,6 +830,31 @@ async def handle_streamable_http(
)
)

# Always mount both /mcp and /mcp/ for full compatibility, regardless of default
_main_path = self.settings.streamable_http_path
if _main_path.endswith("/"):
_alt_path = _main_path.rstrip("/")
else:
_alt_path = _main_path + "/"
if _alt_path != _main_path:
if self._auth_server_provider:
routes.append(
Mount(
_alt_path,
app=RequireAuthMiddleware(
handle_streamable_http,
required_scopes,
),
)
)
else:
routes.append(
Mount(
_alt_path,
app=handle_streamable_http,
)
)

routes.extend(self._custom_starlette_routes)

return Starlette(
Expand Down
Loading