Commit 188cff3
deps: update to http-parser 2.5.2
Fixes http-parser regression with IS_HEADER_CHAR check
Add test case for obstext characters (> 0x80) is header
PR-URL: #5238
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Myles Borins <mborins@us.ibm.com>1 parent 8e592bb commit 188cff3Copy full SHA for 188cff3
File tree
Expand file treeCollapse file tree
5 files changed
+22
-4
lines changedOpen diff view settings
Filter options
- deps/http_parser
- test/parallel
Expand file treeCollapse file tree
5 files changed
+22
-4
lines changedOpen diff view settings
Collapse file
+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
19 | 19 | |
20 | 20 | |
21 | 21 | |
22 | | - |
| 22 | + |
23 | 23 | |
24 | 24 | |
25 | 25 | |
|
Collapse file
deps/http_parser/http_parser.c
Copy file name to clipboardExpand all lines: deps/http_parser/http_parser.c+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
438 | 438 | |
439 | 439 | |
440 | 440 | |
441 | | - |
| 441 | + |
442 | 442 | |
443 | 443 | |
444 | 444 | |
|
Collapse file
deps/http_parser/http_parser.h
Copy file name to clipboardExpand all lines: deps/http_parser/http_parser.h+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
27 | 27 | |
28 | 28 | |
29 | 29 | |
30 | | - |
| 30 | + |
31 | 31 | |
32 | 32 | |
33 | 33 | |
|
Collapse file
+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
3251 | 3251 | |
3252 | 3252 | |
3253 | 3253 | |
3254 | | - |
| 3254 | + |
3255 | 3255 | |
3256 | 3256 | |
3257 | 3257 | |
|
Collapse file
test/parallel/test-http-header-obstext.js
Copy file name to clipboard+18Lines changed: 18 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
0 commit comments