MongoDB NoSQL Injection - Node.js - PullRequest
Новая
       50

MongoDB NoSQL Injection - Node.js

0 голосов
/ 04 октября 2019

Я использую MongoDB 4.2 с Express> 4.

Я пытаюсь выполнить NoSQL Injection, используя node.js. Это код .ejs формы с именем пользователя и паролем: 

 <body>
    <form action="/login_db/login_db" method="POST" >
      <input placeholder="name" name="name"> </br>
      <input placeholder="password" name="password"> </br>
      <input type="submit" value="submit" id="submit">
    </form>
    </body>

У меня есть локальная база данных MongoDB с пользователем, которой разрешено получать доступ через базу данных. Это .js код: 

const MongoClient = require("mongodb").MongoClient;
const ObjectID = require('mongodb').ObjectID;
const dbname = "mydb";

const url = "mongodb://localhost:27017";
const mongoOptions = {useNewUrlParser : true};

const state = {
    db : null
};

const connect = (cb) =>{

    if(state.db)
        cb();
    else{
        MongoClient.connect(url,mongoOptions,(err,client)=>{
            if(err)
                cb(err);
            else{
                state.db = client.db(dbname);
                cb();
            }
        });
    }
}

const getPrimaryKey = (_id)=>{
    return ObjectID(_id);
}

const getDB = ()=>{
    return state.db;
}

module.exports = {getDB,connect,getPrimaryKey};

И это .js login_db-route.js: 

"use strict"
const router = require('express').Router();
const db = require("../db");
const mongoose = require('mongoose');
const passport = require('passport');
const express = require('express');

const collection = "amministrazione";


const app = express()
const path = require('path');

var bodyParser = require('body-parser')



router.get('/login_db', function(req,res){
 console.log('login request');
  var cursor = db.getDB().collection(collection).find().toArray(function(err, results) {
    res.render('login_db.ejs', {quotes: results, user: req.user});
  });

});


router.post('/login_db', function (req, res) {
   console.log('login post request');
  var name = req.body.name;
  var password = req.body.password;
   db.getDB().collection(collection).findOne({"name" : name, "password" : password},  (err, result) =>{
     if(result == null || result.length == 0){
       console.log('user not found '+ err);
       return res.render('login_db.ejs',{user: req.user});
     }
     console.log('User found' + result);
     db.getDB().collection(collection).find().toArray((err, result)=>{
       if(err) return console.log(err)
       res.render('book.ejs', {quotes:result,user: req.user, flag: true});
     })
   })
});


module.exports = router;

код, который я написал, уязвим для внедрения nosql, но когда я пытаюсь вставить в форму полезную нагрузку MongoDB, такую ​​как: 

{ $gt : "a" }
{ $gt : "a" }

, у меня не может быть успешного входа в систему. В чем проблема? Есть ли способ не учитывать "?

Спасибо всем, кто мне помог.

1 Ответ

1 голос
/ 04 октября 2019

Это ваш код. 

var name = req.body.name;
var password = req.body.password;
db.getDB().collection(collection).findOne({"name" : name, "password" : password});

Если вы передадите эти параметры, запрос mongodb будет выглядеть следующим образом. 

db.getDB().collection(collection).findOne({"name" : "{ $gt : \"a\" }", "password" : "{ $gt : \"a\" }"});

Если вы хотите передать эти параметры как запрос, ноне строка, вы можете добавить код, подобный этому. 

var name = req.body.name;
var password = req.body.password;

try {
    name = JSON.parse(name);
} catch (e){}
try {
    password = JSON.parse(password);
} catch (e){}

db.getDB().collection(collection).findOne({"name" : name, "password" : password });
...