Skip to content

Commit 4f7091c

Browse files
Remove "Connection: keep-alive" workaround from domain-task, as it's not needed with .NET Core >= 1.1.0. Fixes aspnet#655
1 parent 66148da commit 4f7091c

File tree

2 files changed

+1
-10
lines changed

2 files changed

+1
-10
lines changed

src/Microsoft.AspNetCore.SpaServices/npm/domain-task/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "domain-task",
3-
"version": "2.0.2",
3+
"version": "2.0.3",
44
"description": "Tracks outstanding operations for a logical thread of execution",
55
"main": "index.js",
66
"scripts": {

src/Microsoft.AspNetCore.SpaServices/npm/domain-task/src/fetch.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,6 @@ function issueRequest(baseUrl: string, req: string | Request, init?: RequestInit
2525
`);
2626
}
2727

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-
3728
return isomorphicFetch(req, init);
3829
}
3930

0 commit comments

Comments
 (0)