Skip to content

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

Rodrigo Muino Tomonari requested to merge github/fork/misterdjules/reused-parser-domain into v8.x-staging
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • [ ] documentation is changed or added
  • commit message follows commit guidelines

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: https://github.com/nodejs/node/issues/25456

Merge request reports