Skip to content

chore: suppress one more url.parse warning#39226

Open
Skn0tt wants to merge 1 commit intomicrosoft:mainfrom
Skn0tt:url-parse-warning
Open

chore: suppress one more url.parse warning#39226
Skn0tt wants to merge 1 commit intomicrosoft:mainfrom
Skn0tt:url-parse-warning

Conversation

@Skn0tt
Copy link
Member

@Skn0tt Skn0tt commented Feb 11, 2026

Closes #38469

@Skn0tt Skn0tt requested a review from dgozman February 11, 2026 15:13
@Skn0tt Skn0tt self-assigned this Feb 11, 2026
@github-actions
Copy link
Contributor

Test results for "tests 1"

6 failed
❌ [playwright-test] › playwright.artifacts.spec.ts:165 › should work with screenshot: only-on-failure @macos-latest-node20
❌ [playwright-test] › playwright.unhandled.spec.ts:19 › should produce uncaughtException when page.route raises @macos-latest-node20
❌ [playwright-test] › playwright.unhandled.spec.ts:35 › should produce unhandledRejection when page.route raises @macos-latest-node20
❌ [playwright-test] › reporter-junit.spec.ts:310 › created › should render existing attachments, but not missing ones @macos-latest-node20
❌ [playwright-test] › reporter.spec.ts:251 › merged › should not have internal error when steps are finished after timeout @macos-latest-node20
❌ [playwright-test] › update-aria-snapshot.spec.ts:669 › update-source-method › should overwrite source when specified in the config @macos-latest-node20

3 flaky ⚠️ [playwright-test] › ui-mode-trace.spec.ts:433 › should work behind reverse proxy `@macos-latest-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@macos-latest-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:812 › should update state on subsequent run `@macos-latest-node20`

38550 passed, 843 skipped


Merge workflow run.

@github-actions
Copy link
Contributor

Test results for "MCP"

3 failed
❌ [chromium] › mcp/roots.spec.ts:47 › check that trace is saved in workspace @mcp-windows-latest
❌ [msedge] › mcp/roots.spec.ts:47 › check that trace is saved in workspace @mcp-windows-latest
❌ [msedge] › mcp/tracing.spec.ts:21 › check that trace is saved with --save-trace @mcp-windows-latest

4776 passed, 135 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]: url.parse() behavior is not standardized and prone to errors that have security implications.

1 participant