Added Timeout to HttpClient to always be longer than the InvocationTimeoutMilliseconds.

ericgreenmix · SteveSandersonMS · commit 372e597f349f · 2017-08-02T09:22:34.000+01:00
HttpClient defaults to 100 seconds. If InvocationTimeoutMilliseconds is greater than the default of the HttpClient, the task will fail with a Task was canceled exception.
diff --git a/src/Microsoft.AspNetCore.NodeServices/HostingModels/HttpNodeInstance.cs b/src/Microsoft.AspNetCore.NodeServices/HostingModels/HttpNodeInstance.cs
@@ -50,6 +50,7 @@ public HttpNodeInstance(NodeServicesOptions options, int port = 0)
                 options.DebuggingPort)
         {
             _client = new HttpClient();
+            _client.Timeout = TimeSpan.FromMilliseconds(options.InvocationTimeoutMilliseconds + 1000);
         }
 
         private static string MakeCommandLineOptions(int port)

Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,7 @@ public HttpNodeInstance(NodeServicesOptions options, int port = 0)`
`50`	`50`	`options.DebuggingPort)`
`51`	`51`	`{`
`52`	`52`	`_client = new HttpClient();`
	`53`	`+ _client.Timeout = TimeSpan.FromMilliseconds(options.InvocationTimeoutMilliseconds + 1000);`
`53`	`54`	`}`
`54`	`55`
`55`	`56`	`private static string MakeCommandLineOptions(int port)`