Error handling когда падает Mongo — как реализовать?

Question

[heducose]

Как сделать вывод ошибки когда база упала? На данный момент у меня просто все крутится, и потом падает весь сервер, если идет запрос в Монгу, а сама Монга не запущена. Использую Mongoose

Мой контроллер:

exports.secret = function (req, res) {
  User.findById(req.user.sub, function (err, user) {
    if (err) {
      res.send('DB is broken')
      return
    }
    if (user) {
      res.send({
        eee: req.user,
        hi: 'SECRET test authorization of registered user'
      })
    } else {
      res.send('user not found')
    }
  })
}

Подозреваю, что я что-то упустил в других частях кода на сервере. Как это исправить, куда смотреть?

И заодно подскажите что почитать на тему error handling? Тяжело мне дается логика этого процесса.

Comments

[Larisa Moroz]

Что вы подразумеваете под «Монга не запущена» — физически отсутствует процесс mongodb на сервере или просто отвалилось соединение через mongoose ?

[heducose]

Larisa Moroz: ну когда я тестирую, то я сам вырубаю Монгу sudo service mongod stop
А что, есть разница?

[heducose]

И дело в том, что все приложение после этого крашиться. Как этого избежать?

[Larisa Moroz]

Без примера вашего кода очень тяжело.
Напишите, как конкретно вы делаете коннект к mongo и как обрабатываете ошибки

[heducose]

Larisa Moroz: я пробовал кучу вариантов. От базового соезидения mongoose.connect(process.env.DATABASE) до вот такого финального варианта:

mongoose.Promise = global.Promise
mongoose.connect(process.env.DATABASE, err => {
  if (err) {
    console.error(err);
    process.exit(1);
  }
})
mongoose.connection.on('error', (err) => {
  console.error(`${err.message}`);
})

[heducose]

Кроме того, я пробовал тот же сценарий вообще в другом приложении - эффект тот же. Вы хотите сказать, что если вы прекращаете процесс Монги и потом делаете запрос через ноду - у вас приложение не крашится? Это ж вроде дефолтное поведение!

[heducose]

Вот смотрите, ничего из этого не работает:

https://stackoverflow.com/questions/20689768/how-t...
https://fatihkalifa.com/prevent-express-from-crash...

[Larisa Moroz]

Не крашится. Пишет в лог ошибку и продолжает себе работать:

options.server = {
	auto_reconnect: true,
	poolSize: 5,
	socketOptions: { keepAlive: 1, connectTimeoutMS: 30000 },
	reconnectTries: 3000
};
var db = mongoose.createConnection('mongodb://localhost/test', options);

mongoose.connection.on('error', function(err) {
	console.log('MONGODB ERROR MONGOOSE LEVEL ',err);
});

db.on('connecting', function() {
	console.info('MONGODB  connecting.');
});

db.on('error', function(err) {
	console.log('MONGODB ERROR ', err);
});

db.on('close', function(err) {
	console.log('MONGODB CLOSE ', err);
});

db.on('connected', function() {
	console.info('MONGODB connected successfully.');
});

db.once('open', function callback() {
	console.info('MONGODB opened successfully.');
});

db.on('reconnected', function() {
	console.info('MONGODB reconnected.');
});

db.on('timeout', function() {
	console.info('MONGODB timeout.');
});

db.on('disconnected', function() {
	console.info('MONGODB disconnected');
});

[heducose]

Не понял, что за mongoose.createConnection, у меня вообще mongoose.connect
Почему такая неконсистенция?
Где best practice посмотреть?
Сейчас вообще швыряет в консоль ошибку депрекейтед open()

[Larisa Moroz]

mongoose.connect создаёт одно дефолтное соединение
mongoose.createConnection используется, когда надо создать несколько соединений, возвращает объект Connection . В доке mongoose есть.

[heducose]

Larisa Moroz: дока у монгуса ужасная, и это не только мое мнение.

Как знать, мне надо одно дефолтное соединение, или несколько?

[Larisa Moroz]

С одним дефолтным на данный момент у меня не получилось отлавливать дисконнекты и настроить переподключение, т.к. mongoose начинает ругаться, что options.server и auto_reconnect deprecated

При использовании createConnection всё прекрасно работает, но для моделей надо будет указывать явно ссылку на объект, возвращаемый createConnection.

Вот пример модели:

const mongoose = require('mongoose');
const db = require('../libs/mongoose');

const TaskSchema = new mongoose.Schema({
  name: String,
  urgency: String
}, {
  timestamps: true,
  toObject: {
    transform(doc, ret) {
      delete ret.__v;
      return ret;
    }
  }
});

module.exports = db.model("Task", TaskSchema)

здесь db — это то, что экспортирует module.exports = db;модуль подключения к базе (libs/mongoose)

[heducose]

Larisa Moroz: спс, попробую этот вариант.