nestjs-redis icon indicating copy to clipboard operation
nestjs-redis copied to clipboard

Some vulnerabilities require your attention to resolve. [High : Server-Side Request Forgery]

Open bug249286 opened this issue 4 years ago • 5 comments

───────────────┬──────────────────────────────────────────────────────────────┐ │ High │ Server-Side Request Forgery │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ axios │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.21.1 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ nestjs-redis │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ nestjs-redis > @nestjs/common > axios │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1594

bug249286 avatar Jan 11 '21 03:01 bug249286

Need to update @nestjs/common. Published npm package is v1.2.8 while the git is v1.3.2. Please publish the latest version to npm.

A workaround - npm rm nestjs-redis and then npm i https://github.com/skunight/nestjs-redis (install directly from git's latest version).

It seems that the package name has changed, so change all the imports to import from nest-redis.

eyalyoli avatar Jan 15 '21 17:01 eyalyoli

@eyalyoli Thanks.

bug249286 avatar Jan 17 '21 15:01 bug249286

OK, since the package name changed, it is now published under the new name which is at https://www.npmjs.com/package/nest-redis

@skunight can you please update the readme that the package name has changed?

eyalyoli avatar Jan 18 '21 11:01 eyalyoli

Did you see fix? But it will be better if it becomes a peer dependency.

rifatdover avatar Jan 18 '21 17:01 rifatdover

@eyalyoli I doubt it was intentionally. There was a PR to revert this change, but it was closed. There is also an open PR that changes the package name back to nestjs-redis. Merging this should resolve the confusion.

@skunight @wisekaa03 please shed the light what is the correct name of the library at this point? Is it going to be further maintenance for nestjs-redis?

It's frustrating to see the project appreciated by the community like this unmaintained.

sashkopavlenko avatar Mar 05 '21 16:03 sashkopavlenko