no-unsafe-query

CWE: CWE-89 OWASP: A03:2021 – Injection

Detects SQL injection in TypeORM raw queries. This rule is part of eslint-plugin-typeorm-security.

💼 This rule is set to error in the recommended config.

Quick Summary

Rule Details

Reports three shapes when they reach a raw-SQL sink:

1. String concatenation — dataSource.query('SELECT ... ' + value)
2. Template interpolation — dataSource.query(`SELECT ... ${value}`)
3. A variable tainted by either, including via +=, then passed to a sink

Sinks

dataSource.query() / manager.query(). Query-builder string fragments (.where("name = '" + x + "'")) are not covered yet — use :name parameters there.

❌ Incorrect

await dataSource.query(`SELECT * FROM users WHERE id = ${userId}`);

await dataSource.query('SELECT * FROM users WHERE email = ' + email);

let sql = 'SELECT * FROM products WHERE 1=1';
sql += ` AND name = '${name}'`;
await dataSource.query(sql);

✅ Correct

await dataSource.query('SELECT * FROM users WHERE id = $1', [userId]);

Known limitations

• Only identifier member access is matched, so dataSource['query'](...) is a false negative.
• Taint tracking is single-scope and name-based — it does not follow a query string across function boundaries.

Implementation

The detection is shared across the driver plugins via createSqlInjectionRule in @interlace/eslint-devkit; this rule supplies TypeORM's sinks and remediation copy. Install the plugin matching your stack and you get exactly one finding per line.

Further Reading

• TypeORM — parameterized queries
• OWASP — SQL Injection
• CWE-89