Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork1.1k
Commit8b3320d
authored
fix: Remove the default connection close header. (#1736)
Instead, we rely on the underlying http implementation in Node.jsto handle this, as per the documentation athttps://nodejs.org/api/http.html#new-agentoptionsThisfixes#1735 and likely replaces#1473The original change introducing this provided no clear motivationfor the override, and the implementation has since been changed todisable this header when an agent is provided, so I think thereis sufficient evidence that removing this is the correct behaviour.af21ae67f685771 parent7b86e94 commit8b3320d
2 files changed
+2
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
539 | 539 | | |
540 | 540 | | |
541 | 541 | | |
542 | | - | |
543 | 542 | | |
544 | 543 | | |
545 | 544 | | |
| |||
558 | 557 | | |
559 | 558 | | |
560 | 559 | | |
| 560 | + | |
| 561 | + | |
561 | 562 | | |
562 | 563 | | |
563 | 564 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
288 | 288 | | |
289 | 289 | | |
290 | 290 | | |
291 | | - | |
292 | | - | |
293 | | - | |
294 | | - | |
295 | 291 | | |
296 | 292 | | |
297 | 293 | | |
| |||
0 commit comments
Comments
(0)