Vuln Fix: Upgrade sequalize to remove Vulnerability - Githubissues

node-casbin / sequelize-adapter

Sequelize adapter for Casbin

https://github.com/casbin/node-casbin

Apache License 2.0

64 stars 34 forks source link

Vuln Fix: Upgrade sequalize to remove Vulnerability #69

Closed js-lowes closed 1 year ago

js-lowes commented 1 year ago

SNYK: https://security.snyk.io/vuln/SNYK-JS-SEQUELIZE-2932027

Fix: https://github.com/node-casbin/sequelize-adapter/issues/70

casbin-bot commented 1 year ago

@nodece @Zxilly @Shivansh-yadav13 please review

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3896463282

5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.795%

Totals
Change from base Build 3105386997:	0.0%
Covered Lines:	113
Relevant Lines:	122

💛 - Coveralls

hsluoyz commented 1 year ago

@js-lowes what's the problem?

js-lowes commented 1 year ago

@js-lowes what's the problem?

@hsluoyz see the latest commit, as I bump up version something is going on: https://github.com/node-casbin/sequelize-adapter/actions/runs/3902541865/jobs/6665722755

hsluoyz commented 1 year ago

Replaced by: https://github.com/node-casbin/sequelize-adapter/pull/77