Permalink
Commits on Feb 15, 2015
  1. @indutny

    header: fix field sizes

    The flags enum does not fit into 6 bits anymore.
    
    Fix: #218
    PR-URL: #219
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    indutny committed Feb 15, 2015
Commits on Jan 26, 2015
  1. @RomainGiraud @indutny

    Add function to initialize http_parser_settings

    PR-URL: #215
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    RomainGiraud committed with indutny Jan 23, 2015
Commits on Jan 25, 2015
  1. @indutny

    Bump version to 2.4.2

    indutny committed Jan 25, 2015
  2. @indutny

    src: skip lws between `connection` values

    Fix: nodejs/node#588
    PR-URL: #216
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    indutny committed Jan 24, 2015
Commits on Jan 14, 2015
  1. @indutny

    Bump version to 2.4.1

    indutny committed Jan 14, 2015
  2. @indutny

    src: fix build on MSVC

    Do explicit casts from/to enums, convert type of `memchr`'s return value
    to `const char*`.
    
    PR-URL: #213
    Reviewed-By: Bert Belder <bertbelder@gmail.com>
    indutny committed Jan 14, 2015
Commits on Jan 13, 2015
  1. @indutny

    Bump version to 2.4.0

    indutny committed Jan 13, 2015
Commits on Dec 28, 2014
  1. @Akagi201 @indutny

    readme: fix typo

    PR-URL: #207
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Akagi201 committed with indutny Dec 28, 2014
Commits on Dec 5, 2014
  1. @indutny

    src: annotate with likely/unlikely

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    indutny committed Dec 1, 2014
  2. @indutny

    bench: add chunked bytes

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    indutny committed Dec 1, 2014
  3. @kolbyjack @indutny

    src: simple Connection header multi-value parsing

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    PR-URL: #100
    kolbyjack committed with indutny Mar 2, 2012
Commits on Nov 29, 2014
  1. @indutny

    src: remove reexecute goto

    indutny committed Nov 28, 2014
  2. @indutny
Commits on Nov 28, 2014
  1. @indutny
  2. @indutny
  3. @indutny
  4. @indutny

    src: save progress

    indutny committed Nov 28, 2014
  5. @indutny

    benchmark: initial

    indutny committed Nov 28, 2014
Commits on Nov 3, 2014
  1. @tjfontaine
Commits on Oct 1, 2014
  1. @alagoutte @indutny

    Fix warning on test suite found by Clang Analyzer

    test.c:2210:9: warning: Value stored to 'char_len' is never read
            char_len = 2;
            ^          ~
    test.c:2921:3: warning: Value stored to 'pass' is never read
      pass &= (parsed == 0);
      ^       ~~~~~~~~~~~~~
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    alagoutte committed with indutny Sep 1, 2014
Commits on Sep 9, 2014
  1. @indutny

    http_parser: Follow RFC-7230 Sec 3.2.4

    RFC-7230 Sec 3.2.4 expressly forbids line-folding in header
    field-names.
    
    This change no longer allows obsolete line-folding between the
    header field-name and the colon. If HTTP_PARSER_STRICT is unset,
    the parser still allows space characters.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Jeff Pinner committed with indutny Aug 20, 2014
  2. @marcomorain @indutny

    Docs fix

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    marcomorain committed with indutny Sep 8, 2014
Commits on Aug 24, 2014
  1. @mmalecki @indutny

    doc: add very basic docs for `http_parser_execute`

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    mmalecki committed with indutny Aug 14, 2014
Commits on Aug 22, 2014
  1. @laggyluke @indutny

    header: typo fix in a comment

    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    laggyluke committed with indutny Aug 21, 2014
Commits on Aug 5, 2014
  1. @alagoutte @indutny

    src: fix clang warning

    Fix http_parser.c:2147:3: warning: Value stored to 'uf' is never
    read found by Clang Analyser.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    alagoutte committed with indutny Aug 4, 2014
Commits on Jun 27, 2014
  1. @helje5 @indutny

    Added support for MKCALENDAR

    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    helje5 committed with indutny Jun 24, 2014
Commits on Jun 21, 2014
  1. @markmontymark @indutny

    very minor spelling/grammar changes in README.md

    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    markmontymark committed with indutny Jun 19, 2014
Commits on Jun 18, 2014
  1. @rlidwka @indutny

    signing the CLA is no longer a requirement

    Signed-off-by: Fedor Indutny <fedor@indutny.com>
    rlidwka committed with indutny Jun 17, 2014
Commits on Apr 10, 2014
  1. @xta @indutny

    fix typo in README comment

    xta committed with indutny Apr 10, 2014
Commits on Mar 19, 2014
  1. @mnemonicflow @indutny
Commits on Mar 17, 2014
  1. @vinniefalco @indutny
Commits on Mar 13, 2014
  1. @indutny

    Bump version to 2.3.0

    indutny committed Mar 13, 2014
Commits on Mar 12, 2014
  1. @dpw @indutny

    Fix issues around multi-line headers

    Always discard leading whitespace in a header value, even if it
    is folded.
    
    Pay attention to values of interesting headers (Connection,
    Content-Length, etc.) even when they come on a continuation line.
    
    Add a test case to check that requests and responses using only LF
    to separate lines are handled correctly.
    dpw committed with indutny Nov 28, 2013
  2. @dpw @indutny

    Include separating ws when folding header values

    The support for folding of multi-line header values does not conform
    to the specs.  Given a request containing
    
        Multi-Line-Header: foo<CRLF>
         bar<CRLF>
    
    http-parser will eliminate the whitespace breaking the header value to
    yield a header value of "foobar".  This is confirmed by the
    LINE_FOLDING_IN_HEADER case in tests.c.
    
    But from rfc2616, section 2.2:
    
       A CRLF is allowed in the definition of TEXT only as part of a header
       field continuation. It is expected that the folding LWS will be
       replaced with a single SP before interpretation of the TEXT value.
    
    And from draft-ietf-httpbis-p1-messaging-25, section 3.2.4:
    
       A server that receives an obs-fold in a request message that is not
       within a message/http container MUST either reject the message by
       sending a 400 (Bad Request), preferably with a representation
       explaining that obsolete line folding is unacceptable, or replace
       each received obs-fold with one or more SP octets prior to
       interpreting the field value or forwarding the message downstream.
    
    So in the example above, the header value should be interpreted as
    "foo bar", possibly with multiple spaces.  The current http-parser
    behaviour of eliminating the LWS altogether clearly deviates from the
    specs.
    
    For http-parser itself to confirm exactly would involve significant
    changes in order to synthesize replacement SP octets.  Such changes
    are unlikely to be worth it to support what is an obscure and
    deprecated feature.  But http-parser should at least preserve some
    separating whitespace when folding multi-line header values, so that
    applications using http-parser can conform to the specs.
    
    This commit is a minimal change to preserve whitespace when folding
    lines.  It eliminates the CRLF, but retains any trailing and leading
    whitespace in the header value.
    dpw committed with indutny Nov 28, 2013
Commits on Jan 28, 2014
  1. @indutny

    Bump version to 2.2.1

    indutny committed Jan 28, 2014