maxHeaderSize It should be an instance property, not a static property of the constructor.

[masx200]

maxHeaderSize It should be an instance property, not a static property of the constructor.

http-parser-js/http-parser.js

Line 33 in 8a13b79

HTTPParser.maxHeaderSize = 80 * 1024; // maxHeaderSize (in bytes) is configurable, but 80kb by default;

[masx200]

https://nodejs.org/dist/latest-v18.x/docs/api/http.html#httpmaxheadersize

This can be overridden for servers and client requests by passing the maxHeaderSize option.

[masx200]

http-parser-js/http-parser.js

Line 138 in 8a13b79

if (this.headerSize > HTTPParser.maxHeaderSize) {

[Jimbly]

I suspect it needs to be both to keep compatibility with both old users and the new Node API.

[masx200]

I suspect it needs to be both to keep compatibility with both old users and the new Node API.

 if  ( this . headerSize  > (this.maxHeaderSize || HTTPParser . maxHeaderSize ))  {