From db56e80b2135dc5dfa87aae5e47df8b67fde2f9c Mon Sep 17 00:00:00 2001 From: Michael Lumish Date: Tue, 18 Jan 2022 12:35:22 -0800 Subject: [PATCH] grpc-js: Add secureConnection error handling in server --- packages/grpc-js/src/server.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/packages/grpc-js/src/server.ts b/packages/grpc-js/src/server.ts index 33057def..9c28a276 100644 --- a/packages/grpc-js/src/server.ts +++ b/packages/grpc-js/src/server.ts @@ -371,6 +371,13 @@ export class Server { creds._getSettings()! ); http2Server = http2.createSecureServer(secureServerOptions); + http2Server.on('secureConnection', (socket: TLSSocket) => { + /* These errors need to be handled by the user of Http2SecureServer, + * according to https://github.com/nodejs/node/issues/35824 */ + socket.on('error', (e: Error) => { + this.trace('An incoming TLS connection closed with error: ' + e.message); + }); + }); } else { http2Server = http2.createServer(serverOptions); }