Permalink
Commits on Jul 17, 2016
  1. @indutny

    Bump version to 2.7.1

    indutny committed Jul 17, 2016
  2. @bnoordhuis @indutny

    parser: fix Content-Length header parsing.

    Commit e2e467b ("Update http-parser to 2.6.1") enforces that messages
    contain no more than one Content-Length header but it considers any
    header that starts with "Content-Length" as a duplicate.
    
    Fix: #324
    PR-URL: #325
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    bnoordhuis committed with indutny Jul 3, 2016
Commits on Jun 23, 2016
  1. @guoxiao @indutny

    test: remove ‘nread’, which is not used

    PR-URL: #300
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    guoxiao committed with indutny Apr 22, 2016
Commits on Jun 1, 2016
  1. @oldratlee @indutny

    readme: improve format, fix code syntax

    PR-URL: #305
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    oldratlee committed with indutny May 4, 2016
Commits on Apr 25, 2016
  1. @indutny

    src: put `assert()` after var declarations

    Microsoft C++ compiler fails when this assert appears before the
    declarations. Moved assert to appear after the declarations.
    
    PR-URL: #293
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Jonathan Marler committed with indutny Mar 15, 2016
Commits on Apr 19, 2016
  1. @indutny

    Bump version to 2.6.0

    indutny committed Apr 19, 2016
  2. @indutny

    parser: returning 2 from on_headers_complete

    Returning `2` from on_headers_complete will tell parser that it
    should not expect neither a body nor any futher responses on
    this connection. This is useful for handling responses to a
    CONNECT request which may not contain `Upgrade` or
    `Connection: upgrade` headers.
    
    See: nodejs/node#6198
    PR-URL: #299
    Reviewed-By: Brian White <mscdex@mscdex.net>
    indutny committed Apr 16, 2016
Commits on Mar 4, 2016
  1. @jbergstroem @jasnell

    test: add a test for for obstext characters (> 0x80)

    Create a test added for nodejs in nodejs/node@954a4b4.
    
    PR-URL: #287
    Reviewed-By: James M Snell <jasnell@gmail.com>
    jbergstroem committed with jasnell Feb 27, 2016
  2. @jbergstroem @jasnell

    Update http-parser to 2.6.2

    Fixes a header parsing bug for obstext characters (> 0x80)
    
    Adaption of nodejs/node@954a4b4:
    
        Author: James M Snell <jasnell@gmail.com>
        Date:   Mon Feb 15 09:40:58 2016 -0800
    
        deps: update to http-parser 2.6.2
    
        Fixes http-parser regression with IS_HEADER_CHAR check
        Add test case for obstext characters (> 0x80) is header
    
        PR-URL: nodejs/node#5237
        Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
        Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
        Reviewed-By: Myles Borins <myles.borins@gmail.com>
    
    PR-URL: #287
    Reviewed-By: James M Snell <jasnell@gmail.com>
    jbergstroem committed with jasnell Feb 27, 2016
Commits on Feb 27, 2016
  1. @dolmen @indutny

    src: refactor method parsing

    Use a switch and a macro to branch parsing of HTTP methods.
    Easier to read and much shorter.
    
    In this commit, the order of branches dispatching is the same as in the
    original code, to ease review. Reordering branches by descending
    frequency will improve speed too.
    
    PR-URL: #273
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    dolmen committed with indutny Oct 12, 2015
Commits on Feb 10, 2016
  1. @jbergstroem @jasnell

    test: Assert against correct error messages

    PR-URL: #279
    Reviewed-By: James M Snell <jasnell@gmail.com>
    jbergstroem committed with jasnell Feb 10, 2016
  2. @jasnell

    Update http-parser to 2.6.1

    Includes parsing improvements to ensure closer HTTP spec conformance
    
    Adaption of nodejs/node@4f4c8ab:
    
        Author: James M Snell <jasnell@gmail.com>
        Date:   Wed Feb 3 17:28:48 2016 -0800
    
        deps: update http-parser to version 2.6.1
    
        includes parsing improvements to ensure closer HTTP spec conformance
    
        PR-URL: nodejs/node-private#26
        Reviewed-By: Rod Vagg <r@va.gg>
        Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
        Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    
    PR-URL: #279
    Reviewed-By: James M Snell <jasnell@gmail.com>
    jasnell committed Feb 10, 2016
Commits on Jan 11, 2016
  1. @geek @indutny

    readme: fix build status badge

    PR-URL: #277
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    geek committed with indutny Jan 7, 2016
Commits on Oct 29, 2015
  1. @jasnell

    Bump version to 2.6.0

    jasnell committed Oct 29, 2015
Commits on Oct 27, 2015
  1. @indutny @jasnell

    src: introduce `http_parser_url_init`

    The struct must be zero-initialized, but this wasn't explicitly stated
    anywhere in headers. Introduce `http_parser_url_init` API method that
    will do it.
    
    Fixes: #209
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Brian White <mscdex@mscdex.net>
    PR-URL: #225
    indutny committed with jasnell Feb 24, 2015
Commits on Oct 26, 2015
  1. @lberezy @jasnell

    doc: updated README.md to include multi-threading example

    Included information and example on communicating data between
    thread local and callback scopes. This shows how http-parser
    can be used in a multi-threaded context.
    
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    PR-URL: #256
    lberezy committed with jasnell Jun 28, 2015
  2. @dolmen @jasnell

    src: support LINK/UNLINK (RFC 2068, draft-snell-link-method)

    Add support for HTTP methods LINK and UNLINK originally defined in RFC2068
    section 19.6.2.2, but with semantic added in a Internet draft.
    https://tools.ietf.org/html/rfc2068#section-19.6.1.2
    https://tools.ietf.org/html/draft-snell-link-method-12
    
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Fedor Induty <fedor@indutny.com>
    PR-URL: #267
    dolmen committed with jasnell Oct 12, 2015
Commits on Jul 23, 2015
  1. @sebastian-philipp @indutny

    src: fixed compile error C2143 for vs2012

    PR-URL: #261
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    sebastian-philipp committed with indutny Jul 22, 2015
  2. @Tatsh @indutny

    header: treat Wine like MinGW

    PR-URL: #259
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Tatsh committed with indutny Jul 12, 2015
Commits on Jul 21, 2015
  1. @Hywan @indutny

    src: support ACL (WebDAV, RFC3744, Section 8.1).

    PR-URL: #260
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Hywan committed with indutny Apr 24, 2015
Commits on Jul 19, 2015
  1. @Hywan @indutny

    readme: update WebSocket link to RFC6455

    PR-URL: #240
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Hywan committed with indutny Apr 24, 2015
Commits on Jul 16, 2015
  1. @Hywan @indutny

    test: `SEARCH`, `PURGE` and `MKCALENDAR`

    These methods are defined in `http_parser.h` but not tested.
    
    PR-URL: #241
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Hywan committed with indutny Apr 24, 2015
  2. @Hywan @indutny

    src: support BIND/REBIND/UNBIND (WebDAV, RFC5842)

    Support BIND/REBIND/UNBIND methods, see sections 4, 5, 6 of
    RFC5842.
    
    PR-URL: #242
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Hywan committed with indutny Apr 24, 2015
Commits on Jul 5, 2015
  1. @tatsuhiro-t @indutny

    src: support IPv6 Zone ID as per RFC 6874

    IPv6 scoped address contains Zone ID, which is appended to IPv6
    address after '%' separator.  RFC 6874 says that Zone ID after '%'
    must consist of 1*(unreserved or pct-encoded).  This commit adds this
    IPv6 Zone ID support.
    
    PR-URL: #253
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    tatsuhiro-t committed with indutny Jun 18, 2015
Commits on Jul 4, 2015
  1. @deleisha @indutny

    src: use ARRAY_SIZE instead of sizeof()

    PR-URL: #252
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    deleisha committed with indutny Jun 12, 2015
Commits on May 21, 2015
  1. @jscissr @indutny

    src: remove double check

    This is already included in `http_message_needs_eof`.
    
    PR-URL: #248
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    jscissr committed with indutny May 20, 2015
Commits on May 19, 2015
  1. @OnixGH @indutny

    src: fix invalid memory access in http_parse_host

    http_parse_host() depends on `u->field_data[UF_HOST]`, but this
    if() allowed the method to be called if only
    `u->field_data[UF_SCHEMA]` was set, resulting in use of
    unintialized pointers.
    
    PR-URL: #246
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    OnixGH committed with indutny May 18, 2015
  2. @guoxiao @indutny
Commits on Apr 24, 2015
  1. @indutny

    Bump version to 2.5.0

    indutny committed Apr 24, 2015
Commits on Apr 23, 2015
  1. @KjellSchubert @indutny

    src: support body in Upgrade requests

    Invoke message_complete cb for upgrade with body.
    
    (D1364677 + D1380182 orig author afrind@fb.com)
    
    Fix: #234
    PR-URL: #235
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    KjellSchubert committed with indutny Apr 17, 2015
  2. @KjellSchubert @indutny

    src: callbacks chunk boundaries: header/complete

    (Proxygen fork merge D508755 D521404, orig author simpkins@fb.com)
    
    Fix: #231
    PR-URL: #233
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    KjellSchubert committed with indutny Apr 18, 2015
  3. @KjellSchubert @indutny

    test: regression test for incomplete/corrupted hdr

    Add regression test that verifies that old bugs regarding corrupted
    and incomplete headers are fixed.
    
    (D512233 + D570451, orig authors brianp@fb.com + jtarnawski@fb.com)
    
    PR-URL: #236
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    KjellSchubert committed with indutny Apr 17, 2015
Commits on Mar 16, 2015
  1. @Umorrian @indutny

    makefile: add un/install targets

    Add install, install-strip and uninstall targets, make DESTDIR usable.
    
    PR-URL: #228
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Umorrian committed with indutny Mar 7, 2015
Commits on Feb 24, 2015
  1. @jay @indutny

    url_parser: remove mixed declarations

    (dump_url)
    - Remove erroneous quote in output.
    
    (main)
    - Remove mixed declarations for compatibility with pre-c99 compilers
    like msvc.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    PR-URL: #224
    jay committed with indutny Feb 24, 2015
Commits on Feb 19, 2015
  1. @indutny

    src: partially revert 959f4cb to fix nread value

    With 959f4cb on reexecution - header byte was accounted twice, leading
    to the incorrect `parser->nread` value.
    
    Fix: #220
    PR-URL: #221
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    indutny committed Feb 19, 2015