Skip to content

Conversation

@yury-s
Copy link
Member

@yury-s yury-s commented Oct 14, 2025

new URL() adds / after the domain if:

  • the scheme is a special scheme per WHATWG spec: http, https, ftp, file, ws, wss, and
  • no explicit path was provided.

Aftter this PR: preserve explicit scheme to avoid inadvertent slash. Treat any scheme pattern as a special scheme for the slash resolution.

Fixes #37821

@github-actions
Copy link
Contributor

Test results for "MCP"

2550 passed, 102 skipped


Merge workflow run.

@yury-s yury-s merged commit 3f67e71 into microsoft:main Oct 14, 2025
34 of 35 checks passed
@yury-s yury-s deleted the glob-shema branch October 14, 2025 17:29
@github-actions
Copy link
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › runner.spec.ts:118 › should ignore subprocess creation error because of SIGINT @macos-latest-node18-1

3 flaky ⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node18`
⚠️ [firefox-page] › page/page-event-request.spec.ts:182 › should return response body when Cross-Origin-Opener-Policy is set `@firefox-ubuntu-22.04-node18`
⚠️ [chromium-library] › library/proxy.spec.ts:93 › should proxy local network requests › with other bypasses › link-local `@ubuntu-22.04-chromium-tip-of-tree`

47017 passed, 816 skipped


Merge workflow run.

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.

[Bug]: waitForRequest With Custom Protocol Stopped Working

2 participants