Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: add --max-http-header-size flag #24811

Merged
merged 3 commits into from Dec 20, 2018
Merged

Commits on Dec 20, 2018

  1. deps: cherry-pick http_parser_set_max_header_size

    This commit adds http_parser_set_max_header_size() to the
    http-parser for overriding the compile time maximum HTTP
    header size.
    
    PR-URL: nodejs#24811
    Fixes: nodejs#24692
    Refs: nodejs/http-parser#453
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
    Reviewed-By: Myles Borins <myles.borins@gmail.com>
    Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
    Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
    cjihrig committed Dec 20, 2018
    Copy the full SHA
    cdcb1b7 View commit details
    Browse the repository at this point in the history
  2. src: add kUInteger parsing

    This commit adds support for uint64_t option parsing.
    
    PR-URL: nodejs#24811
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Myles Borins <myles.borins@gmail.com>
    Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
    Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
    mcollina authored and cjihrig committed Dec 20, 2018
    Copy the full SHA
    436f4de View commit details
    Browse the repository at this point in the history
  3. cli: add --max-http-header-size flag

    Allow the maximum size of HTTP headers to be overridden from
    the command line.
    
    co-authored-by: Matteo Collina <hello@matteocollina.com>
    PR-URL: nodejs#24811
    Fixes: nodejs#24692
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Myles Borins <myles.borins@gmail.com>
    Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
    Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
    cjihrig and mcollina committed Dec 20, 2018
    Copy the full SHA
    632f263 View commit details
    Browse the repository at this point in the history