Ecosyste.ms: Advisories

Security Advisories: MDE2OlNlY3VyaXR5QWR2aXNvcnlHSFNBLTRxaHgtZzl3cC1nOW02

Failure to sanitize quotes which can lead to sql injection in squel

All versions of squel are vulnerable to sql injection.

The squel package does not properly escape user provided input when provided using the setFields method. This could lead to sql injection if the query was then executed.

Proof of concept demonstrating the injection of a single quote into a generated sql statement from user provided input.

> console.log(squel.insert().into('buh').setFields({foo: "bar'baz"}).toString());
INSERT INTO buh (foo) VALUES ('bar'baz')

Recommendation

There is no fix at this time and the issue has been reported publicly. Consider using another query builder that provides strong guarantees for input sanitization to prevent sql injection attacks.

Permalink: https://github.com/advisories/GHSA-4qhx-g9wp-g9m6
JSON: https://advisories.ecosyste.ms/api/v1/advisories/MDE2OlNlY3VyaXR5QWR2aXNvcnlHSFNBLTRxaHgtZzl3cC1nOW02
Source: GitHub Advisory Database
Origin: Unspecified
Severity: Critical
Classification: General
Published: almost 5 years ago
Updated: over 1 year ago

Identifiers: GHSA-4qhx-g9wp-g9m6
References:

• https://github.com/hiddentao/squel/issues/350
• https://www.npmjs.com/advisories/575
• https://github.com/advisories/GHSA-4qhx-g9wp-g9m6

Repository: https://github.com/hiddentao/squel
Blast Radius: 0.0

Affected Packages