Skip to content

chore: continuing with request to axios changes #31915

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jun 27, 2025

Conversation

tgriesser
Copy link
Member

Part of #31540, re: encryption layer

Additional details

Follow on to #31905, adding the ability to configure encryption when constructing a new CloudRequest client

Steps to test

How has the user experience changed?

PR Tasks

@tgriesser tgriesser changed the title fix: normalize options passed via axios to the agent layer chore: continuing with request to axios changes Jun 20, 2025
Base automatically changed from tgriesser/fix/fix-axios-proxy-networking to develop June 23, 2025 18:52
@tgriesser tgriesser force-pushed the tgriesser/fix/fix-axios-client-encryption branch from 1845a33 to 79d707f Compare June 24, 2025 15:16
@tgriesser tgriesser marked this pull request as ready for review June 25, 2025 18:26
@tgriesser tgriesser requested a review from ryanthemanuel June 25, 2025 18:26
Copy link

cypress bot commented Jun 26, 2025

cypress    Run #63378

Run Properties:  status check passed Passed #63378  •  git commit 2421898aa6: cleanup from PR review
Project cypress
Branch Review tgriesser/fix/fix-axios-client-encryption
Run status status check passed Passed #63378
Run duration 09m 34s
Commit git commit 2421898aa6: cleanup from PR review
Committer Tim Griesser
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 694
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 130
View all changes introduced in this branch ↗︎

@tgriesser tgriesser requested a review from ryanthemanuel June 27, 2025 20:07
Copy link
Collaborator

@ryanthemanuel ryanthemanuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks!

@tgriesser tgriesser merged commit ed8e7ea into develop Jun 27, 2025
90 of 91 checks passed
@tgriesser tgriesser deleted the tgriesser/fix/fix-axios-client-encryption branch June 27, 2025 21:18
ryanthemanuel pushed a commit that referenced this pull request Jun 30, 2025
* chore: updating v8 snapshot cache

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
ryanthemanuel pushed a commit that referenced this pull request Jun 30, 2025
* chore: updating v8 snapshot cache

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
ryanthemanuel pushed a commit that referenced this pull request Jun 30, 2025
* chore: updating v8 snapshot cache

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

* index on develop: ed8e7ea chore: continuing with request to axios changes (#31915)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jul 1, 2025

Released in 14.5.1.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Jul 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants