Skip to content

async_hooks: add executionAsyncResource

Rodrigo Muino Tomonari requested to merge github/fork/Qard/current-resource into master

This is a rebase of #21313

With llhttp in Node.js, the http parser reuse code now gives each reuse an AsyncResource so the concerns of the original attempt at a "current resource" API are no longer valid, unblocking it from landing now. This is basically identical to the original PR with some minor changes to appease the linter and to shuffle around some bits that moved in timers refactoring.

I'm working on this so I can update #26540 to use this form of the current resource API, eliminating a potential memory leak concern.

cc @nodejs/diagnostics @nodejs/async_hooks

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

Merge request reports