Skip to content
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

Warning about deprecated sub-dep 'uuid' when installing cypress #17622

Closed
dapinto8 opened this issue Aug 5, 2021 · 4 comments · Fixed by #17813
Closed

Warning about deprecated sub-dep 'uuid' when installing cypress #17622

dapinto8 opened this issue Aug 5, 2021 · 4 comments · Fixed by #17813
Labels
topic: installation Issue during installation or downloading Cypress

Comments

@dapinto8
Copy link

dapinto8 commented Aug 5, 2021

Current behavior

warning cypress > @cypress/request > [email protected]: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.

Desired behavior:

No warning

Cypress Version

8.2.0

Package Manager

yarn

Operating system

Linux

@dapinto8 dapinto8 added the topic: installation Issue during installation or downloading Cypress label Aug 5, 2021
@JessicaSachs
Copy link
Contributor

JessicaSachs commented Aug 18, 2021

@flotwig we should move off of @cypress/request since request was put into maintenance mode and won't be accepting any fixes. Thoughts?

@flotwig
Copy link
Contributor

flotwig commented Aug 19, 2021

@JessicaSachs I agree. We use the fork at https://github.com/cypress-io/request but it is limiting since there's no support for HTTP2, QUIC, retries, etc... It would improve our code base and make the proxy layer faster if we switched to a modern request lib that speaks HTTP/2/3.

Related issue on HTTP/2: #3708

We can avoid this deprecation issue today by upgrading the dep in @cypress/request.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Aug 23, 2021

The code for this is done in cypress-io/cypress#17813, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot cypress-bot bot removed the stage: needs review The PR code is done & tested, needs review label Aug 23, 2021
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Aug 27, 2021

Released in 8.3.1.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Aug 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic: installation Issue during installation or downloading Cypress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants