DEV Community

Ofri Peretz
Ofri Peretz

Posted on • Originally published at ofriperetz.dev

Getting Started with eslint-plugin-mongodb-security

MongoDB stores JavaScript objects. Your query is already structured data — there is no "query string" to inject into. Which is exactly why NoSQL injection looks different from SQL injection, and why generic security linters miss it.

The attack isn't ; DROP TABLE users; --. It's this:

// POST body: { "username": "admin", "password": { "$ne": null } }
await db.collection("users").findOne({
  username: req.body.username,
  password: req.body.password,  // ← operator injection bypasses auth
});
Enter fullscreen mode Exit fullscreen mode

eslint-plugin-mongodb-security is the only ESLint plugin built specifically for MongoDB/Mongoose codebases. Here's how to use it.


Install

npm install eslint-plugin-mongodb-security --save-dev
Enter fullscreen mode Exit fullscreen mode

eslint.config.mjs:

import mongodbSecurity from "eslint-plugin-mongodb-security";

export default [
  {
    plugins: { "mongodb-security": mongodbSecurity },
    rules: mongodbSecurity.configs.flagship.rules,
  },
];
Enter fullscreen mode Exit fullscreen mode

The three rules you need most

1. no-unsafe-query — NoSQL operator injection (CWE-943, CVSS 9.8)

Fires when a $where, $expr, or $function operator receives a value directly from user input — the exact pattern that lets an attacker inject arbitrary query logic.

// ❌ Flagged — $where with user-controlled JavaScript
db.collection("orders").find({
  $where: `this.total > ${req.query.minTotal}`,
});
Enter fullscreen mode Exit fullscreen mode
// ✅ Safe — use $gt instead of $where
db.collection("orders").find({
  total: { $gt: Number(req.query.minTotal) },
});
Enter fullscreen mode Exit fullscreen mode

2. no-operator-injection — Query operator in request body (CWE-943, CVSS 9.1)

When req.body (or any request property) is used directly in a MongoDB query field, an attacker can send { "$ne": null } or { "$gt": "" } as the field value to bypass authentication or extract unauthorized data.

// ❌ Flagged — req.body.password could be { "$ne": null }
const user = await User.findOne({
  email: req.body.email,
  password: req.body.password,
});
Enter fullscreen mode Exit fullscreen mode
// ✅ Safe — hash and compare separately
const user = await User.findOne({ email: req.body.email });
const valid = await bcrypt.compare(req.body.password, user.passwordHash);
Enter fullscreen mode Exit fullscreen mode

3. no-hardcoded-connection-string — Credentials in source (CWE-798, CVSS 7.5)

Detects mongodb:// and mongodb+srv:// connection strings with embedded credentials in source code. These get committed to git history and exposed in build artifacts.

// ❌ Flagged — credentials in source
const client = new MongoClient(
  "mongodb+srv://admin:hunter2@cluster0.example.com/mydb"
);
Enter fullscreen mode Exit fullscreen mode
// ✅ Safe — from environment variable
const client = new MongoClient(process.env.MONGODB_URI);
Enter fullscreen mode Exit fullscreen mode

Why a MongoDB-specific plugin

Generic security linters (eslint-plugin-security, eslint-plugin-sonarjs) don't know the MongoDB query API. They can't distinguish db.collection("users").find({ $where: userInput }) from console.log({ $where: "debug" }). The MongoDB-specific plugin knows:

  • Which methods are query execution points (.find(), .findOne(), .aggregate(), .updateMany(), etc.)
  • Which operators are dangerous ($where, $expr, $function, $accumulator)
  • What constitutes user input in the MongoDB context

All 16 rules

Rule Severity CWE
no-unsafe-query error CWE-943
no-operator-injection error CWE-943
no-hardcoded-connection-string error CWE-798
no-hardcoded-credentials error CWE-798
require-tls-connection error CWE-319
require-auth-mechanism warn CWE-306
no-unsafe-regex-query error CWE-1333
no-unsafe-where error CWE-943
no-debug-mode-production warn CWE-489
require-schema-validation warn
no-select-sensitive-fields warn CWE-312
no-bypass-middleware warn CWE-284
no-unsafe-populate warn CWE-943
no-unbounded-find warn CWE-400
require-projection warn
require-lean-queries warn

If this catches something in your codebase, ⭐ star the repo — it keeps the rules maintained.


npm · Rule docs · ⭐ GitHub

Top comments (0)