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.

Sign up for GitHub

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

Jump to bottom

http: improve ClientRequest constructor #10654

Merged
merged 6 commits into from
Jan 13, 2017
Merged

http: improve ClientRequest constructor #10654

merged 6 commits into from
Jan 13, 2017

Commits on Jan 13, 2017

  1. http: optimize short path validation 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    Configuration menu
    Copy the full SHA
    5c2ef14 View commit details
    Browse the repository at this point in the history

  2. http: optimize default method case 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    Configuration menu
    Copy the full SHA
    dab22b5 View commit details
    Browse the repository at this point in the history

  3. http: avoid duplicate isArray() 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    Configuration menu
    Copy the full SHA
    15c6187 View commit details
    Browse the repository at this point in the history

  4. http: misc ClientRequest cleanup 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    1 Configuration menu
    Copy the full SHA
    2f77596 View commit details
    Browse the repository at this point in the history

  5. benchmark: add ClientRequest creation benchmark 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    Configuration menu
    Copy the full SHA
    c5d0fd9 View commit details
    Browse the repository at this point in the history

  6. test: add http.ClientRequest defaults test 

    PR-URL: nodejs#10654
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
    @mscdex
    mscdex committed Jan 13, 2017
    Configuration menu
    Copy the full SHA
    57e2ec4 View commit details
    Browse the repository at this point in the history