Восстановление после Uncaught Exception в Node.JS

Question

ОК, у меня проблема. Если во время обработки HTTP-запроса возникает необработанное исключение, у меня нет возможности вызвать метод end() объекта http.ServerResponse. Поэтому сервер зависает навсегда и никогда не выполняет запрос.

Вот пример:

var express = require('express');
var app = express.createServer();
var reqNum = 0;
app.get('/favicon.ico', function(req, res) {res.send(404);});
app.get('*', function(req, res, next) {
    console.log("Request #", ++reqNum, ":", req.url);
    next();
});
app.get('/error', function(req, res, next) {
    throw new Error("Problem occurred");
});
app.get('/hang', function(req, res, next) {
    console.log("In /hang route");
    setTimeout(function() {
        console.log("In /hang callback");
        if(reqNum >= 3)
            throw new Error("Problem occurred");
        res.send("It worked!");
    }, 2000);
});
process.on('uncaughtException', function(err) {
    console.log("Uncaught exception!", err);
});
app.listen(8080);

Если вы посещаете / ошибка, возникает исключение, но оно ловится. Пользователь получает сообщение об ошибке - нет проблем. Однако, если я зайду / повисну, сервер в конечном итоге выдаст необработанное исключение и зависнет навсегда. Любые последующие запросы на / зависать будут зависать.

Это отстой. Любой совет, как решить эту проблему?

Answer 1

Когда возникает необъяснимое исключение, вы находитесь в нечистом состоянии. Позвольте процессу умереть и перезапустить его, вы ничего не можете сделать, чтобы безопасно вернуть его в заведомо хорошее состояние. использование forever, он перезапустит ваш процесс, как только он умрет.

Answer 2

Если ошибка генерируется синхронно, экспресс не перестает работать, возвращая только 500.

this.app.get("/error", (request, response) => {
  throw new Error("shouldn't stop");
});

Если ошибка генерируется асинхронно, экспресс завершится сбоем. Но, согласно официальной документации, все еще есть способ восстановиться, позвонив next:

this.app.get("/error", (request, response, next) => {
  setTimeout(() => {
    try {
      throw new Error("shouldn't stop");
    } catch (err) {
      next(err);
    }
  }, 0);
});

Это позволит express выполнить свою обязанность ответить с ошибкой 500.

Answer 3

Используйте try/catch/finally.

app.get('/hang', function(req, res, next) {
    console.log("In /hang route");
    setTimeout(function() {
        console.log("In /hang callback");
        try {
            if(reqNum >= 3)
                throw new Error("Problem occurred");
        } catch (err) {
            console.log("There was an error", err);
        } finally {
            res.send("It worked!");
        }
    }, 2000);
});