fix(auth): normalize default resource URL ports - #3299
Conversation
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
|
Thanks for the contribution. This repository only keeps pull requests open when they're linked to an issue that a maintainer has assigned to the author — CONTRIBUTING.md explains why and how we work. This PR has been closed for now because you aren't currently assigned to #3297. If a maintainer would like this change as a PR from you, they'll assign you to #3297 and this PR will reopen automatically — there's nothing more you need to do. (If you opened the issue, this PR already shows up on its timeline.) There's no need to open a new PR — this one will be reopened. While it's closed, please push any updates as new commits rather than force-pushing, since GitHub can't reopen a PR whose branch has been rewritten. Maintainers: reopening this PR, removing the |
Summary
Fixes #3297
Testing
uv run --frozen pytest tests/shared/test_auth_utils.py(19 passed)uv run --frozen pyright src/mcp/shared/auth_utils.py tests/shared/test_auth_utils.py(0 errors)uv run --frozen ruff check .uv run --frozen ruff format --check .bash scripts/testcould not run.AI assistance
AI assistance was used during investigation and implementation. I reviewed the change, reproduced the issue, and understand the code and tests being submitted.