react-localize-redux icon indicating copy to clipboard operation
react-localize-redux copied to clipboard

Vulnerabilities in the latest react-localize-redux

Open johnwen84 opened this issue 2 years ago • 8 comments

Do you want to request a feature or report a bug? bug

What is the current behavior? npm i react-localize-redux@latest

5 high severity vulnerabilities

To address all issues (including breaking changes), run: npm audit fix --force

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code. Paste the link to your JSFiddle (https://jsfiddle.net/Luktwrdm/) or CodeSandbox (https://codesandbox.io/s/new) example below: Just run, npm i react-localize-redux@latest npm will report 5 high severity vulnerabilities

What is the expected behavior? We expect that no vulnerability should be reported We can run "npm audit fix --force" to get rid of the vulnerabilities, but that will bring the version to 2.17.5, which causes package conflicts and maybe more other problems.

Which versions of react and react-localize-redux are you using? "react": "^16.8.4", "react-cookie": "^4.0.3", "react-dom": "^16.8.4", "react-localize-redux": "^3.5.3", "react-redux": "^7.2.0", "react-router-dom": "^5.2.0", "redux": "^4.0.1", "redux-thunk": "^2.3.0",

johnwen84 avatar Mar 29 '22 14:03 johnwen84

It's a shame, but the library doesn't seem maintained anymore.

cristianoccazinsp avatar May 09 '22 22:05 cristianoccazinsp

@cristianoccazinsp yea it sucks. I guess I'll have to find an alternative.

johnwen84 avatar May 11 '22 22:05 johnwen84

Let me know if you find any!

cristianoccazinsp avatar May 11 '22 22:05 cristianoccazinsp

@ryandrewjohnson Any change you can update the package to fix these? Using this heavily in production

jimmailcamp avatar May 16 '22 10:05 jimmailcamp

Bump!

cristianoccazinsp avatar Aug 12 '22 15:08 cristianoccazinsp

Just switched to i18nreact. Def recommend it. You can opt out for the route prefixes easily. Basicly just wrap the t function in i18n with your own component. And change every import statement where you used to use this package. Worth the hassle, as it's very maintained unlike this shitshow.

jimmailcamp avatar Aug 15 '22 11:08 jimmailcamp