http-parser
http-parser copied to clipboard
http request/response parser for c
Hello, I tried packaging this library for OpenIndiana. Would it be possible for you to have the following changes to the Makefile? 1. Change `LIBDIR = $(PREFIX)/lib` to `LIBDIR ?=...
My reading of rfc2616 is that both spaces and horizontal tabs are allowed as LWS in field values: ``` LWS = [CRLF] 1*( SP | HT ) ``` But in...
rfc7230 defines: ``` reason-phrase = *( HTAB / SP / VCHAR / obs-text ) ``` `VCHAR` excludes all control characters except HTAB, but http-parser allows them: https://github.com/nodejs/http-parser/blob/f2c26ee500ab3921010fa7ec66243365611e77dd/http_parser.c#L916
The strict definition of status-line is: ``` status-line = HTTP-version SP status-code SP reason-phrase CRLF ``` But the http-parser allows an unlimited number of spaces before status-code: https://github.com/nodejs/http-parser/blob/f2c26ee500ab3921010fa7ec66243365611e77dd/http_parser.c#L873 Shouldn't this...
rfc7230 defines asterisk-form: ``` asterisk-form = "*" ``` But it seems `parse_url_char` allows additional characters after reading an asterisk. Note how state changes to `s_req_path` after encountering an asterisk at...
From rfc3986: ``` userinfo = *( unreserved / pct-encoded / sub-delims / ":" ) pct-encoded = "%" HEXDIG HEXDIG ``` However, `parse_url_char` accepts a percent symbol followed by any userinfo...
There are some comments in http_parser.cc, line 1837: /\* Here we call the headers_complete callback. This is somewhat \* different than other callbacks because if the user returns 1, we...
I've implemented a [simple C++ wrapper](https://github.com/AndreLouisCaron/httpxx) for the `http-parser` library and ran into a problem when using the parser pause feature. Basically, I have a `Request` object that wraps a...
The parser does not support non-alpha characters in the scheme of a URI. This is supported as part of the [spec for URIs](https://tools.ietf.org/html/rfc3986#section-3.1).
I'm creating this issue on behalf of the Proxygen team, to gauge the odds of the corresponding Proxygen fork's changeset being accepted back into http_parser, quoting Proxygen task description: >...