[app] ignore errors from accessing disconnected client sockets

Technically `EHOSTUNREACH` and `ETIMEDOUT` should go into a
 'disconnected_read' metric. But this would require changes to
 dashboards and a larger diff.
This commit is contained in:
Jakob Ackermann 2020-08-12 11:45:36 +01:00
parent 8f7d846ed9
commit a0028646a5

View file

@ -181,7 +181,11 @@ if (Settings.shutdownDrainTimeWindow) {
if (Settings.errors && Settings.errors.catchUncaughtErrors) {
process.removeAllListeners('uncaughtException')
process.on('uncaughtException', function (error) {
if (['EPIPE', 'ECONNRESET'].includes(error.code)) {
if (
['ETIMEDOUT', 'EHOSTUNREACH', 'EPIPE', 'ECONNRESET'].includes(
error.code
)
) {
Metrics.inc('disconnected_write', 1, { status: error.code })
return logger.warn(
{ err: error },