You're opening a new database connection on every request. Here's one way you could rearrange your code to create one connection and one Users
object, and use that later in the request handler:
var http = require('http');
var mongoose = require('mongoose');
// we want Users to be in the global scope here, not in a function
// that way different functions can both see the same variable
var Users = null;
mongoose.connect('mongodb://localhost/bookmarks');
var db = mongoose.connection;
db.on('error', console.error.bind(console, 'connection error:'));
// By placing Users and createServer inside the db.once('open') callback,
// we wait till mongo is ready before letting the http handler query users:
db.once('open', function(){
console.log('Running');
// now overwrite the already defined Users variable:
Users = mongoose.model('users', { name: String, lastname: String, yas: Number, yer:String });
http.createServer(function (req, res) {
findUsers(function(err, results){
if(err){
res.writeHead(500);
res.end();
} else {
res.writeHead(200, {'Content-Type': 'application/json; charset=utf-8'});
res.end(results);
};
});
}).listen(1337,127.0.0.1);
});
function findUsers(callback){
Users.find().lean().exec(function(err,users) {
// console.log(JSON.stringify(users));
var sonuc=JSON.stringify(users);
console.log(sonuc);
//res.end(sonuc);
callback(err, sonuc);
});
}