Passing Node.js parameter into mongodb request

Question

I'm working on a simple web service in node.js. I'm using choreographer to route the http calls. This code works fine:

router.get('/search/*', function(req, res, term){
  res.writeHead(200, {'Content Type:':'text/plain'});
  db.collection('foo').find({'a':1}).toArray(function(err, items){
  console.log(items);
    res.write(JSON.stringify(items));
    res.end();
   });
  });

As you can see, the find method looks for {'a':1}, this works fine, a record is returned. But when I want to pass the search term from the router to the query I get a null response:

router.get('/search/*', function(req, res, term){
  res.writeHead(200, {'Content Type:':'text/plain'});
  db.collection('foo').find({'a':term}).toArray(function(err, items){
  console.log(items);
    res.write(JSON.stringify(items));
    res.end();
   });
  });

Any ideas anyone??

Edit: I have checked the value of term, as suggested below in the comments, it is 1, which is what I expected it to be.

Have you checked the value of err? If there are any problems executing the query, err will be non-null and items will be null. It's good to always check. — mpobrien
– mpobrien, Commented Dec 5, 2011 at 20:53

Rickard Zachrisson · Accepted Answer · 2011-12-05 19:48:51Z

1

Could it be that the db-connection takes longer than the actually routing?

If you define "term" and get a reponse it is most likley the term is not found in db or the db-connection fails in some way.

A way could be to init and call db from a callback function.

router.get('/search/*', function(req, res, term){
  res.writeHead(200, {'Content Type:':'text/plain'});
  var db = new mongo.Db('dbname', server);
  db.open(function(err, db){
    db.createCollection("collection_name", function(err, collection){
      db.collection('foo').find({'a':term}).toArray(function(err, items){
        console.log(items);
      });
    });
  });
});

If it work you may want to throw in a db.close();

Reservation for bad syntax.

answered Dec 5, 2011 at 19:48

Rickard Zachrisson

9857 silver badges11 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Twobard Over a year ago

'term' (1) is definitely in the database, the variable 'term' has been initialized correctly when the database is queried

Twobard · Accepted Answer · 2011-12-06 01:36:44Z

1

I found a workaround was to build up a new json object then set the 'a' parameter to term afterwards e.g.

var searchterm = {'a':2};
searchterm.a = term;

I suspect there is something about creating JSON objects i'm not fully understanding here.

edited Dec 6, 2011 at 1:36

answered Dec 5, 2011 at 21:56

Twobard

2,5831 gold badge24 silver badges27 bronze badges

Collectives™ on Stack Overflow

Passing Node.js parameter into mongodb request

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related