Commit e64f2f5
committed
doc: fix incorrect default for maxHeadersCount
request.maxHeadersCount and server.maxHeadersCount were documented
with a default of 2000. The underlying parser limit,
MAX_HEADER_PAIRS in lib/_http_common.js, is a count of name/value
pairs (2 slots per header), so the effective maximum number of
headers is 1000, not 2000.
Verified against a running server: sending 1000 headers returns all
of them, while sending 1001 headers still returns only 1000, with
the extra header silently dropped.
Fixes: #65470
Signed-off-by: suraj-shrivastav <suraj.shrivastavvv@gmail.com>1 parent 0b89f8f commit e64f2f5
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1127 | 1127 | | |
1128 | 1128 | | |
1129 | 1129 | | |
1130 | | - | |
| 1130 | + | |
1131 | 1131 | | |
1132 | 1132 | | |
1133 | 1133 | | |
| |||
1914 | 1914 | | |
1915 | 1915 | | |
1916 | 1916 | | |
1917 | | - | |
| 1917 | + | |
1918 | 1918 | | |
1919 | 1919 | | |
1920 | 1920 | | |
| |||
0 commit comments