search

cockroachdb / sequelize-cockroachdb

Use Sequelize and CockroachDB together
Apache License 2.0
53 stars 16 forks source link

Sanitize version string #126

Closed RichardJCai closed 3 years ago

RichardJCai commented 3 years ago

Using replacements in the sequelize.query function is safe since the string is escaped.

https://github.com/cockroachdb/cockroach/issues/66500 Fixes Sequelize part here.

RichardJCai commented 3 years ago

nice!

so looks like we should wait for #127 to merge, then create a new 6.0.1 release? (i can do the 6.0.1 release if you'd like)

Yep, sounds right.