fix: decode urls in prerequest

[mschile]

Additional details

When correlating prerequests, it's possible one of them will be decoded and the other will not. This will lead to an unnecessary correlation timeout of two seconds.

This occurs because we use the legacy url API while CDP uses the WHATWG URL standard. #28369 has been logged to update the codebase to the WHATWG URL API.

However, in the meantime, we are decoding all prerequests as they come in so the correlations will be correct.

The following requests were observed in the wild:

With the encoded | as %7C:
https://fonts.googleapis.com/css?family=Roboto+Mono%7COpen+Sans:700&display=swap

Already decoded |:
https://fonts.googleapis.com/css?family=Roboto+Mono|Open+Sans:700&display=swap

Steps to test

How has the user experience changed?

PR Tasks

• Have tests been added/updated?
• [n/a] Has a PR for user-facing changes been opened in cypress-documentation?
• [n/a] Have API changes been updated in the type definitions?

[cypress]

4 flaky tests on run #52601 ↗︎

0

6062

77

0

4

Details:

adding server integration tests
Project: cypress	Commit: def0e214d4
Status: Passed	Duration: 14:16 💡
Started: Nov 29, 2023 7:18 PM	Ended: Nov 29, 2023 7:32 PM

  e2e/origin/snapshots.cy.ts • 1 flaky test • 5x-driver-electron

View Output

Test		Artifacts
cy.origin - snapshots > e2e log verification > Does not take snapshots of XHR/fetch requests from secondary origin if the wrong origin is visited / origin mismatch, but instead the primary origin (existing behavior)		Test Replay

  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-electron

View Output

Test		Artifacts
... > correctly returns currentRetry		Test Replay
... > correctly returns currentRetry		Test Replay
... > correctly returns currentRetry		Test Replay

Review all test suite changes for PR #28427 ↗︎

[cypress-bot]

Released in 13.6.1.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.6.1, please open a new issue.