fix: app crashing on websocket upgrade request over https

[ryanthemanuel]

• Closes Cypress crashes on websocket upgrade request over https after update from 9.7.0 to 10.0.3 #22217

User facing changelog

App no longer crashes on websocket upgrade request over https

Additional details

A change was made on on #20107 to support fetch requests in the proxy. For that change, we needed to set options.uri in our HttpsAgent.addRequest method. It was assumed that this would cover the existing logic to set options.uri as well, however, this didn't cover the case where we have a client certificate over https. This PR restores that functionality.

Steps to test

You can test this by uncommenting the code change and running the new test and see that there is a failure. You can also use the test project provided by @AtofStryker here

How has the user experience changed?

The app no longer crashes when there is

PR Tasks

• Have tests been added/updated?
• Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
• [na] Has a PR for user-facing changes been opened in cypress-documentation?
• [na] Have API changes been updated in the type definitions?

[cypress-bot]

Thanks for taking the time to open a PR!

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.
• Become familiar with the Code Review Checklist for guidelines on coding standards and what needs to be done before a PR can be merged.

[cypress]

---

Test summary

41474 • 0 • 1434 • 0 • 15

---

Run details

Project	cypress
Status	Passed
Commit	34b52d5
Started	Aug 19, 2022 7:37 PM
Ended	Aug 19, 2022 7:52 PM
Duration	15:03 💡
OS	Linux Debian - 11.3
Browser	Multiple

View run in Cypress Dashboard ➡️

---

Flakiness

	cypress/e2e/commands/net_stubbing.cy.ts		5
1	network stubbing > intercepting request > can add a body to a request that does not have one
2	network stubbing > intercepting request > can delay with deprecated delayMs param
3	network stubbing > intercepting request > can delay with deprecated delayMs param
4	network stubbing > intercepting request > can delay with deprecated delayMs param
5	... > with `times` > only uses each handler N times
This comment includes only the first 5 flaky tests. See all 15 flaky tests in the Cypress Dashboard.

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[rachelruderman]

Nice fix!