Remove "Connection: keep-alive" workaround from domain-task, as it's not needed with .NET Core >= 1.1.0. Fixes aspnet#655

SteveSandersonMS · SteveSandersonMS · commit 4f7091c5d5e9 · 2017-02-09T14:28:18.000Z
diff --git a/src/Microsoft.AspNetCore.SpaServices/npm/domain-task/package.json b/src/Microsoft.AspNetCore.SpaServices/npm/domain-task/package.json
@@ -1,6 +1,6 @@
 {
   "name": "domain-task",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "description": "Tracks outstanding operations for a logical thread of execution",
   "main": "index.js",
   "scripts": {
diff --git a/src/Microsoft.AspNetCore.SpaServices/npm/domain-task/src/fetch.ts b/src/Microsoft.AspNetCore.SpaServices/npm/domain-task/src/fetch.ts
@@ -25,15 +25,6 @@ function issueRequest(baseUrl: string, req: string | Request, init?: RequestInit
         `);
     }
 
-    // Currently, some part of ASP.NET (perhaps just Kestrel on Mac - unconfirmed) doesn't complete
-    // its responses if we send 'Connection: close', which is the default. So if no 'Connection' header
-    // has been specified explicitly, use 'Connection: keep-alive'.
-    init = init || {};
-    init.headers = init.headers || {};
-    if (!init.headers['Connection']) {
-        init.headers['Connection'] = 'keep-alive';
-    }
-
     return isomorphicFetch(req, init);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "domain-task",`
`3`		`- "version": "2.0.2",`
	`3`	`+ "version": "2.0.3",`
`4`	`4`	`"description": "Tracks outstanding operations for a logical thread of execution",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"scripts": {`
Original file line number	Diff line number	Diff line change
`@@ -25,15 +25,6 @@ function issueRequest(baseUrl: string, req: string \| Request, init?: RequestInit`
`25`	`25`	`);
`26`	`26`	`}`
`27`	`27`
`28`		`- // Currently, some part of ASP.NET (perhaps just Kestrel on Mac - unconfirmed) doesn't complete`
`29`		`- // its responses if we send 'Connection: close', which is the default. So if no 'Connection' header`
`30`		`- // has been specified explicitly, use 'Connection: keep-alive'.`
`31`		`- init = init \|\| {};`
`32`		`- init.headers = init.headers \|\| {};`
`33`		`- if (!init.headers['Connection']) {`
`34`		`- init.headers['Connection'] = 'keep-alive';`
`35`		`- }`
`36`		`-`
`37`	`28`	`return isomorphicFetch(req, init);`
`38`	`29`	`}`
`39`	`30`