From a9bc1201d145c672228ece9b5bb3a598764bcad5 Mon Sep 17 00:00:00 2001 From: SteveSandersonMS Date: Fri, 20 Jan 2017 18:09:57 +0000 Subject: [PATCH] Fix the build by passing allowConnectionDraining from SocketNodeInstance --- .../SocketNodeInstance.cs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/Microsoft.AspNetCore.NodeServices.Sockets/SocketNodeInstance.cs b/src/Microsoft.AspNetCore.NodeServices.Sockets/SocketNodeInstance.cs index f64be2d..b89d5a4 100644 --- a/src/Microsoft.AspNetCore.NodeServices.Sockets/SocketNodeInstance.cs +++ b/src/Microsoft.AspNetCore.NodeServices.Sockets/SocketNodeInstance.cs @@ -62,11 +62,15 @@ namespace Microsoft.AspNetCore.NodeServices.Sockets { if (_connectionHasFailed) { + // _connectionHasFailed implies a protocol-level error. The old instance is no longer of any use. + var allowConnectionDraining = false; + // This special exception type forces NodeServicesImpl to restart the Node instance throw new NodeInvocationException( "The SocketNodeInstance socket connection failed. See logs to identify the reason.", - null, - nodeInstanceUnavailable: true); + details: null, + nodeInstanceUnavailable: true, + allowConnectionDraining: allowConnectionDraining); } if (_virtualConnectionClient == null)