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

Build/Deprecation warnings #48

Open
jaguardo opened this issue Aug 11, 2021 · 9 comments
Open

Build/Deprecation warnings #48

jaguardo opened this issue Aug 11, 2021 · 9 comments

Comments

@jaguardo
Copy link
Contributor

Started looking at all the Warning when building with react-ros:

seems there are currently 5 components: docz, roslib, react-scripts, microbundle-crl, uws:
Docz:

warning docz > docz-core > wait-on > [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142                                                                
warning docz > docz-core > wait-on > @hapi/[email protected]: Switch to 'npm install joi'                                                                                                                  
warning docz > gatsby > @hapi/[email protected]: Switch to 'npm install joi'                                                                                                                               
warning docz > gatsby > [email protected]: babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.                                                                
warning docz > gatsby > [email protected]: This loader has been deprecated. Please use eslint-webpack-plugin                                                                                      
warning docz > gatsby > gatsby-cli > @hapi/[email protected]: Switch to 'npm install joi'                                                                                                                  
warning docz > gatsby > [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.                                                                                                                                                                         
warning docz > gatsby-theme-docz > gatsby-source-filesystem > better-queue > [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.                                                                                                                    
warning docz > docz-core > wait-on > 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.                                                                                                                                                  
warning docz > gatsby > gatsby-cli > [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.                                                                                                                                                            
warning docz > gatsby > gatsby-telemetry > [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.                                                                                                                                                      
warning docz > gatsby > webpack-dev-server > [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                                                  
warning docz > gatsby > webpack-dev-server > chokidar > [email protected]: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.                            
warning docz > docz-core > wait-on > request > [email protected]: this library is no longer supported                                                                                             
warning docz > docz-core > wait-on > @hapi/joi > @hapi/[email protected]: Moved to 'npm install @sideway/address'                                                                                       
warning docz > docz-core > wait-on > @hapi/joi > @hapi/[email protected]: This version has been deprecated and is no longer supported or maintained                                                      
warning docz > docz-core > wait-on > @hapi/joi > @hapi/[email protected]: This version has been deprecated and is no longer supported or maintained                                                        
warning docz > docz-core > wait-on > @hapi/joi > @hapi/[email protected]: This version has been deprecated and is no longer supported or maintained                                                        
warning docz > docz-core > wait-on > @hapi/joi > @hapi/topo > @hapi/[email protected]: This version has been deprecated and is no longer supported or maintained                                           
warning docz > gatsby > gatsby-cli > gatsby-recipes > @hapi/[email protected]: This version has been deprecated and is no longer supported or maintained                                                   
warning docz > gatsby > gatsby-cli > gatsby-recipes > @hapi/[email protected]: Switch to 'npm install joi'                                                                                                 
warning docz > gatsby > gatsby-cli > gatsby-recipes > [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.                                                                                                                                           
warning docz > gatsby > webpack-hot-middleware > [email protected]: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.                                  
warning docz > gatsby-theme-docz > gatsby-plugin-mdx > static-site-generator-webpack-plugin > url > [email protected]: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.                                                                                                                                                                                   
warning docz > gatsby > @pmmmwh/react-refresh-webpack-plugin > native-url > [email protected]: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.       
warning docz > gatsby > webpack-dev-server > sockjs > [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.                                                                                                                                           
warning docz > gatsby > webpack-dev-server > webpack-log > [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.                                                                                                                                      
warning docz > gatsby-theme-docz > gatsby-plugin-mdx > remark > unified > @types/vfile > @types/[email protected]: This is a stub types definition. vfile-message provides its own type definitions, so you do not need this installed.                                                                                                                                                               
warning docz > gatsby > webpack > watchpack > watchpack-chokidar2 > [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                           
warning docz > gatsby-theme-docz > babel-plugin-export-metadata > @babel/cli > @nicolo-ribaudo/chokidar-2 > braces > snapdragon > source-map-resolve > [email protected]: https://github.com/lydell/resolve-url#deprecated                                                                                                                                                                              
warning docz > gatsby-theme-docz > babel-plugin-export-metadata > @babel/cli > @nicolo-ribaudo/chokidar-2 > braces > snapdragon > source-map-resolve > [email protected]: Please see https://github.com/lydell/urix#deprecated 

Which seems to jiv with that packages issues... (doczjs/docz#1568) so, either start to PR changes to that repo, or find another package that does something similar? not familiar, but might not have a long life span...

roslib:

warning roslib > socket.io > [email protected]: Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)                                                                                                                                       
warning roslib > [email protected]: Deprecated due to CVE-2021-21366 resolved in 0.5.0 

socket.io (RobotWebTools/roslibjs#322) looks like they depend on 4.0.0 which is what the warning says not to use... maybe a PR with them?

microbundle-crl > [email protected]: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.

no idea

react-scripts

warning react-scripts > [email protected]: babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.                                                                 
warning react-scripts > [email protected]: This loader has been deprecated. Please use eslint-webpack-plugin                                                                                      
warning react-scripts > [email protected]: "Please update to latest v2.3 or v2.2"                                                                                                                      
warning react-scripts > webpack-dev-server > [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                                                  
warning react-scripts > webpack-dev-server > sockjs > [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.                                                                                                                                           
warning react-scripts > [email protected]: please switch to a stable version                                                                                                        
warning react-scripts > jest-environment-jsdom-fourteen > jsdom > [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142                                   
warning react-scripts > workbox-webpack-plugin > workbox-build > @hapi/[email protected]: Switch to 'npm install joi'                                                                                      
warning react-scripts > babel-jest > @jest/transform > jest-haste-map > [email protected]: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.            
warning react-scripts > jest-environment-jsdom-fourteen > jsdom > [email protected]: request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142                                                                                                                                                    
warning react-scripts > resolve-url-loader > rework > css > [email protected]: Please see https://github.com/lydell/urix#deprecated                                                                        
warning react-scripts > babel-jest > @jest/transform > jest-haste-map > [email protected]: some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features addedwarning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142            
warning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > [email protected]: request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142                                                                                                                             
warning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > [email protected]: use String.prototype.padStart()                                                            
warning react-scripts > workbox-webpack-plugin > workbox-build > strip-comments > babel-plugin-transform-object-rest-spread > babel-runtime > [email protected]: core-js@<3.3 is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.                           

might just have to accept these, as there are about 1000 issues (https://github.com/facebook/create-react-app/issues)
and this one

warning [email protected]: New code is available at github.com/uNetworking/uWebSockets.js  

looks like they are up to version 19ish now (https://github.com/uNetworking/uWebSockets.js)

this is more just to document them and see if there are good alternatives or ways to address them

@jaguardo
Copy link
Contributor Author

jaguardo commented Aug 11, 2021

Docz might be back on track soon:? doczjs/docz#1634
however, is it a package that needs to be in the release?

@jaguardo
Copy link
Contributor Author

jaguardo commented Aug 11, 2021

looks like

"uws": "^10.148.2"

in package.json should work

@flynneva
Copy link
Owner

i should hve some time next week to work on updating the react-ros package. good job at tracking this though! if you find anything else that needs updating let me know and ill add it to the list for next week

@jaguardo
Copy link
Contributor Author

will do... not sure how much you have looked at performance (google Chrome f12), but I've never really looked too hard at "dropped frames" until now. I think it is the way the messages are handled, I'm trying to minimize the demand, but I still get dropped frames every now and then... a little judder here and there. I have a feeling it is within roslibjs and the way the listener.subscribe is created. Interested if other folks have dived in that far, or "if it works, it works!" which is about where I am at the moment!

@flynneva
Copy link
Owner

keep an eye at what Foxglove studios are doing. they are doing some really good work IMO with regards to web visualization stuff.
https://github.com/foxglove

@flynneva
Copy link
Owner

what im trying to say is if you are seeing a problem most likely they are too. ill bet money they are either working on a fix or coming up with a new way around the problem

@jaguardo
Copy link
Contributor Author

oh nice... I'll give them a look!

@flynneva
Copy link
Owner

@jaguardo I just pushed an update to fix some of these warnings. a lot of them are sort of out of my hands though....so if you'd like to keep pushing for updates on other repos feel free 😄 I respect a lot the work you are doing tbh, it takes someone very determined to keep pushing for these sorts of updates on all these repos! 😅

@flynneva flynneva reopened this Aug 23, 2021
@flynneva
Copy link
Owner

going to leave this open to track all these updates. docz build is failing due to some missing deps i think.

Can't resolve 'uws' in '/home/flynn/code/web/ros/react-ros/node_modules/engine.io/lib'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants