fix: handle malformed URIs in prerequests

[mschile]

• Closes URIError: URI malformed in Cypress 13.6.1 #28521

Additional details

In #28427 we started decoding URIs during the prerequest logic to resolve some correlation timeouts. However, we are not correctly handling malformed URIs which will cause Cypress to crash. This change tries to decode the URI and if it fails, just falls back to using the URI as-is for correlation.

Steps to test

How has the user experience changed?

Cypress no longer crashes when receiving a malformed URI.

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]

6 flaky tests on run #52797 ↗︎

0

5371

80

0

6

Details:

moved tryDecodeURI
Project: cypress	Commit: 43456a6829
Status: Passed	Duration: 14:32 💡
Started: Dec 14, 2023 1:29 PM	Ended: Dec 14, 2023 1:43 PM

  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-chrome:beta

View Output

Test		Artifacts
network stubbing > waiting and aliasing > yields the expected interception when two requests are raced		Test Replay

  commands/storage.cy.ts • 1 flaky test • 5x-driver-chrome:beta

View Output

Test		Artifacts
src/cy/commands/storage > #getAllLocalStorage > gets local storage from all origins		Test Replay

  commands/files.cy.js • 1 flaky test • 5x-driver-chrome:beta

View Output

Test		Artifacts
... > throws a specific error when file exists when it shouldn't		Test Replay

  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-chrome:beta

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 #28522 ↗︎

[cypress-bot]

Released in 13.6.2.

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