sequlize 3.9.0 security/injection issue

marin-liovic / restabase

REST interface for your SQL database.

MIT License

43 stars 1 forks source link

Closed williamn closed 8 years ago

williamn commented 8 years ago

Hi,

Any particular reason why you still require sequalize version 3.9.0?

I got this when I run npm install

npm WARN deprecated sequelize@3.9.0: security/injection issue with limit/offset fixed in 3.17.0

I'll be happy to send a PR to update the version.

Thanks.

marin-liovic commented 8 years ago

Hey, I think it should be fine to use any 3.x version. Feel free to create a PR.

Thanks, Marin

williamn commented 8 years ago

OK I will go to make the PR.

How can I run all the tests in one commannd to make I don't break anything?

Thx

marin-liovic commented 8 years ago

You can run all tests with npm test. Sadly they do not cover whole functionality yet...

williamn commented 8 years ago