Commit 1bb2967
http: fix non-string header value concatenation
Since headers are stored in an empty literal object ({}) instead
of an object created with Object.create(null), care must be taken
with property names inherited from Object. Currently there are
only functions inherited, so we can safely check for existing
strings instead.
Fixes: #4456
PR-URL: #4460
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>1 parent 437d0e3 commit 1bb2967Copy full SHA for 1bb2967
File tree
Expand file treeCollapse file tree
2 files changed
+6
-2
lines changedOpen diff view settings
Filter options
- lib
- test/parallel
Expand file treeCollapse file tree
2 files changed
+6
-2
lines changedOpen diff view settings
Collapse file
+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
159 | 159 | |
160 | 160 | |
161 | 161 | |
162 | | - |
| 162 | + |
163 | 163 | |
164 | 164 | |
165 | 165 | |
|
Collapse file
test/parallel/test-http-server-multiheaders.js
Copy file name to clipboardExpand all lines: test/parallel/test-http-server-multiheaders.js+5-1Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
16 | 16 | |
17 | 17 | |
18 | 18 | |
| 19 | + |
19 | 20 | |
20 | 21 | |
21 | 22 | |
| ||
48 | 49 | |
49 | 50 | |
50 | 51 | |
51 | | - |
| 52 | + |
| 53 | + |
| 54 | + |
| 55 | + |
52 | 56 | |
53 | 57 | |
54 | 58 | |
0 commit comments