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

[v8.x] http: attach reused parser to correct domain #25459

Closed
wants to merge 3 commits into from
Closed

[v8.x] http: attach reused parser to correct domain #25459

wants to merge 3 commits into from

Commits on Jan 11, 2019

  1. http: attach reused parser to correct domain 

    Reused parsers can be attached to the domain that corresponds to the
active domain when the underlying socket was created, which is not
necessarily correct.

Instead, we attach parsers to the active domain if there is one when
they're reused from the pool.

Refs: nodejs#25456
    Julien Gilli committed Jan 11, 2019
    Copy the full SHA
    fdcb150 View commit details
    Browse the repository at this point in the history

Commits on Jan 14, 2019

  1. changes according to latest code review

    Julien Gilli committed Jan 14, 2019
    Copy the full SHA
    c1004ad View commit details
    Browse the repository at this point in the history

  2. get rid of dead code

    Julien Gilli committed Jan 14, 2019
    Copy the full SHA
    2e459d0 View commit details
    Browse the repository at this point in the history