Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

mongodb nodejs - converting circular structure

I have some code that pulls all documents from a collection and puts it onto a webpage. a simplified version looks like this:

var mongodb = require("mongodb"),
    express = require("express"),
    mongoServer = new mongodb.Server('localhost', 27017),
    dbConnector = new mongodb.Db('systemMonitor', mongoServer),
    db;

var app = new express();

app.get('/drives', function(req, res) {
  db.collection('driveInfo', function(err, collection) {
    if (err) throw err;
    collection.find({}, function(err, documents) {
      res.send(documents);
    });
  });
});

dbConnector.open(function(err, opendb) {
  if (err) throw err;
  db = opendb;
  app.listen(80);
});

I have a driveInfo collection which contains a long list of documents. Each document contains nested objects. What I would like to do, is whenever someone visits /drives in their browser, to print the entire collection as a json object so that I can grab everything with jquery later (beginnings of an api)

However, I get an error saying "TypeError: Converting circular structure to JSON". The error on the page points to this line of code:

collection.find({}, function(err, documents) {
  res.send(documents);
});

I'm unsure what the problem is, or where the self-reference is. Am I not querying the collection properly?

like image 962
Simon Avatar asked Feb 25 '15 09:02

Simon


2 Answers

Not sure what version of the API you are using, but i think that your syntax might be wrong looking at the API spec:

http://docs.mongodb.org/manual/reference/method/db.collection.find/

This is the declaration:

db.collection.find(<criteria>, <projection>)

And you are definitely misusing the projection parameter. Passing a callback like you are doing seems to return the db object in the result, which is causing the circular error during JSON serialization in express.

The correct code for the find all operation should be something like:

collection.find({}).toArray(function(error, documents) {
    if (err) throw error;

    res.send(documents);
});
like image 59
Faris Zacina Avatar answered Nov 19 '22 15:11

Faris Zacina


In my case I was getting the error because I was querying(using mongoose find method) without doing an await. Please see below

Query that gave the error (as I haven't executed this query using await) :

const tours = Tour.find({
    startLocation: {
      $geoWithin: { $centerSphere: [[longitude, latitude], radius] }
    }
  });

Error that I got on postman due to this :

"message": "Converting circular structure to JSON\n    --> starting at object with constructor 'NativeTopology'\n    |     property 's' -> object with constructor 'Object'\n    |     property 'sessionPool' -> object with constructor 'ServerSessionPool'\n    --- property 'topology' closes the circle"

How I got rid of the above error (added await) :

 const tours = await Tour.find({
        startLocation: {
          $geoWithin: { $centerSphere: [[longitude, latitude], radius] }
        }
      });
like image 5
utkarsh-k Avatar answered Nov 19 '22 16:11

utkarsh-k