Public API for starting in the headers-state about http-parser HOT 6 CLOSED

It's a reasonable request but I don't think it's possible to implement.

Multipart body parts are separated by a boundary marker that http-parser would need to scan for. The parser doesn't backtrack so a partial false positive could lead to data being dropped, particularly if the message is spread out over several chunks. To illustrate:

HTTP/1.1 200 OK
Content-Type: multipart/mixed; boundary="marker"

---marker
Content-Type: text/plain

foo
---mar<BREAK>quee
bar
---marker--

The <BREAK> here is the chunk boundary. The parser has seen (but not stored!) ---mar in the previous chunk. When it starts parsing the second chunk, it discovers it's not actually parsing a boundary marker but by then it's too late.

from http-parser.

I'm not suggesting that http-parser would scan for boundaries. On the contrary; I want to scan for boundaries. I would create a new http-parser and only give it this:

Content-Type: text/plain

foo

from http-parser.

Right, that would be fairly easy to implement but it doesn't seem like there's a need for it. Create a new parser, feed it a trivial HTTP/1.1 200 OK status line, then feed it the multipart body, done.

from http-parser.

Agreed; closing.

from http-parser.

Feeding it a status line is indeed a simple workaround, but a bit of a hacky solution. Would you accept a PR for this?

from http-parser.

@tdecaluwe I don't think that it is a good idea. Through recent years, we have refactored http-parser pretty significantly internally, and exposing these states will lock us into a particular API just for this use-case.

Sorry, but won't fix.

from http-parser.