Почему может падать сервер на Node.js?

Question

[OcelotJungle]

Использую ExpressJS. HTTP сервер запускаю через модуль http, HTTPS сервер запускаю через модуль spdy, чтобы HTTP2, само приложение на Next.js, использую API Routes, запускаю по схеме "Custom Server", в файле server.js есть обработчик запросов к динамическим изображениям:

const dynamicFileHandler = async (req, res, next) => {
    try {
        const filePath = path.join(process.cwd() + decodeURIComponent(req.baseUrl));
        await fs.promises.access(filePath);
        res.sendFile(filePath);
    } catch(e) { next(); }
}
express.use("/images/*", dynamicFileHandler);

Все остальные запросы обрабатываются этим фрагментом:

express.all("/api/*", nextHandler);
express.get("*", nextHandler);

И вот случается иногда такое, что приложение просто падает с таким сообщением:

net.js:817
bytes += el.chunk instanceof Buffer ?
^

TypeError: Cannot read property 'chunk' of null
at Socket.bytesWritten (net.js:817:17)
at Socket.socketOnError (_http_server.js:637:31)
at Socket.emit (events.js:327:22)
at Stream.onStreamError (/root/app/node_modules/spdy/lib/spdy/handle.js:110:18)
at Stream.emit (events.js:315:20)
at errorOrDestroy (/root/app/node_modules/spdy-transport/node_modules/readable-stream/lib/internal/streams/destroy.js:98:101)
at onwriteError (/root/app/node_modules/spdy-transport/node_modules/readable-stream/lib/_stream_writable.js:424:5)
at onwrite (/root/app/node_modules/spdy-transport/node_modules/readable-stream/lib/_stream_writable.js:450:11)
at WritableState.onwrite (/root/app/node_modules/spdy-transport/node_modules/readable-stream/lib/_stream_writable.js:160:5)
at doWrite (/root/app/node_modules/spdy-transport/node_modules/readable-stream/lib/_stream_writable.js:409:30)

Я пока не смог понять, при каких условиях и как это повторить, происходит в случайные моменты, но не спонтанно, а при запросе (что, впрочем, логично, учитывая стак ошибки) от клиента.
В данный момент хочу попробовать разобраться, к чему относится эта строка в net.js, и выяснить, моя это ошибка или какой-то неприятный баг в самом Node. Можно повесить автозапуск, но это костыль, хочется решить по-человечески.
Кто-нибудь может помочь выяснить, где может скрываться проблема?

Comments

[Михаил]

Ошибка именно на запрос изображений?
Ловите на process.on('uncaughtException' и process.on('unhandledRejection'

[OcelotJungle]

Михаил, в том и проблема, что я понятия не имею, в какой момент и из-за чего возникает ошибка, и даже не могу воспроизвести. Если из-за обработчика запросов к картинкам, то есть два спорных момента: req.baseUrl и res.sendFile, т.к. ошибка связана с сетью. С другой стороны, всё это обёрнуто в try-catch и, вроде бы, не должно приводить к падению приложения, даже если что-то пошло не так.

[Михаил]

OcelotJungle, асинхронный выбивается из try/catch, ловите ошибки на процессе

[OcelotJungle]

Михаил, заметил, что, вообще-то, всё это обёрнуто в один большой try-catch с выводом ошибки в консоль и process.exit(1), видимо, это и происходит. Добавил try-catch ко всем обработчикам запросов (даже forceSSL), может поймаю.
Разве асинхронное не ловится try-catch, если используется async/await синтаксис?

[Михаил]

OcelotJungle, on 'uncaughtException' и 'unhandledRejection' карл

Answer 1

[rPman]

открываешь файл net.js строка 817
bytes += el.chunk instanceof Buffer
и добавляешь перед этой строчкой (аккуратно выбери место) проверку el на null, и вызываешь обработку ошибки

Comments

[OcelotJungle]

Не думаю, что лезть руками в файлы Node.js - хорошая идея. На Гитхаб не зальёшь, какое-нибудь обновление потрёт ручные правки. Разве что если поправить культурно не удастся.

[rPman]

это файл модулей? почему пути тогда нет

не важно, очевидно что либо ошибка в модуле (маловероятно но бывает) либо неправильные параметры в используемых методах, так всегда бывает когда не проверяешь на ошибки каждый вызов, способный и выдать

[OcelotJungle]

rPman, net.js - это вот этот net.js: https://github.com/nodejs/node/blob/master/lib/net..., 806 строка.

[rPman]

хм, это кажется метод как бы колбек на пересылаемые данные в сокет
проверяй, где то отправляешь вместо буфера с данными - null

ничто не мешает на время поиска проблемы подсунуть в этот код логирование ошибки (с дампом стека), проверяешь, если !el то дампишь в лог файл содержимое (new Error()).stack

[OcelotJungle]

rPman, спасибо за совет. Не смог докопаться до самого net.js, запихнул в файлы spdy, может поймаю чего. Понять бы, что вызывает ошибку.
Я сам буферы не отправляю, даже не умею с ними работать, нуб ещё. Мой предел - это res.write.

[rPman]

где то в СВОЕМ коде отправляя сообщение вместо строки посылаешь null