Skip to content

Node 16.2 regression, crashing with node::InternalCallbackScope::Close() error #38815

Closed
Closed
Node 16.2 regression, crashing with node::InternalCallbackScope::Close() error#38815
Labels
async_hooksIssues and PRs related to the async hooks subsystem.

Description

@gaggle
gaggle
opened on May 26, 2021

Upgrading from Node 16.1 to 16.2 causes Node to crash, and I don't know how to debug the problem further. This is related to promises and maybe async_hooks or cls-hooked library, but I'm not knowledgeable enough to understand how or why it actually happens.

  • Version: v16.2.0
  • Platform: Darwin 20.4.0 Darwin Kernel Version 20.4.0: Thu Apr 22 21:46:47 PDT 2021; root:xnu-7195.101.2~1/RELEASE_X86_64 x86_64
  • Subsystem:

What steps will reproduce the bug?

I have cooked up a minimal repo here: https://github.com/heap-dk/node-16.2-crash/

The problem manifests itself somehow in relation to promises and async hooks... I think! The repo's readme shows the problem, the code is as minimal as I could make it to showcase the problem, and the commands to replicate the issue are simple. I hope that's good enough to be of interest to you.

The repo has dependencies on dd-trace and cls-hooked, but I don't think the problem is unique to those dependencies, rather I think it's related to their underlying use of async_hooks and somehow it results in a crash... 👀 But I'm guessing on those details so I very much hope someone with more knowledge about these systems can see the deeper pattern!

How often does it reproduce? Is there a required condition?

100% crash.

What is the expected behavior?

I didn't expect a Node patch bump to introduce a breaking behavior like this.

What do you see instead?

node[90090]: ../src/api/callback.cc:125:void node::InternalCallbackScope::Close(): Assertion `(env_->execution_async_id()) == (0)' failed.
 1: 0x10dde07e5 node::Abort() (.cold.1) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 2: 0x10ca93b39 node::Abort() [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 3: 0x10ca939a1 node::Assert(node::AssertionInfo const&) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 4: 0x10c9c3cdd node::InternalCallbackScope::Close() [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 5: 0x10c9c35de node::InternalCallbackScope::~InternalCallbackScope() [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 6: 0x10ca5f4a3 node::StartExecution(node::Environment*, std::__1::function<v8::MaybeLocal<v8::Value> (node::StartExecutionCallbackInfo const&)>) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 7: 0x10c9c81fd node::LoadEnvironment(node::Environment*, std::__1::function<v8::MaybeLocal<v8::Value> (node::StartExecutionCallbackInfo const&)>) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 8: 0x10cad53f0 node::NodeMainInstance::Run(node::EnvSerializeInfo const*) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
 9: 0x10ca6197d node::Start(int, char**) [/Users/jon/.nvm/versions/node/v16.2.0/bin/node]
10: 0x7fff20653f3d start [/usr/lib/system/libdyld.dylib]
11: 0x2 
error Command failed with signal "SIGABRT".

Additional information

To be honest I don't understand the finer points of this crash, but I'm creating this issue because my perception is that upgrading from Node 16.1 to 16.2 introduces this regression. I'm happy to add more context or try different solutions if that will be helpful.

And I want to take a moment to thank the maintainers of Node for making these releases available, crashing or not I'm grateful for your hard work.

Activity

  1. gaggle
    changed the title [-]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close(): Assertion `(env_->execution_async_id()) == (0)' failed`[/-] [+]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close(): Assertion (env_->execution_async_id()) == (0)' failed`[/+] on May 26, 2021
  2. gaggle
    changed the title [-]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close(): Assertion (env_->execution_async_id()) == (0)' failed`[/-] [+]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close(): Assertion (env_->execution_async_id()) == (0) failed`[/+] on May 26, 2021
  3. gaggle
    changed the title [-]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close(): Assertion (env_->execution_async_id()) == (0) failed`[/-] [+]Node 16.2 regression, crashing with `node::InternalCallbackScope::Close()` error[/+] on May 26, 2021
  4. Ayase-252
    added
    async_hooksIssues and PRs related to the async hooks subsystem.
    on May 26, 2021

  5. aduh95 commented on May 26, 2021

    @aduh95
    aduh95
    on May 26, 2021
    Contributor

    A repro that doesn't depend on any npm package would be very helpful.

  6. Ayase-252 commented on May 26, 2021

    @Ayase-252
    Ayase-252
    on May 26, 2021
    Member

    Reproduciable in 16.2.0 but not in master. It may have been fixed. I'd retest it when 16.3.0 is released.

  7. hongbo-miao commented on May 27, 2021

    @hongbo-miao
    hongbo-miao
    on May 27, 2021

    Met same issue. Another demo to help reproduce at #38814

  8. gaggle commented on May 28, 2021

    @gaggle
    gaggle
    on May 28, 2021
    Author

    A repro that doesn't depend on any npm package would be very helpful.

    Just to say I did try to remove the two dependencies, but I couldn't find a way to still trigger the error. I suspect there is a way if I better understood the underlying trigger, but given my current understanding I couldn't find it. Sorry, I do appreciate external dependencies are not the responsibility of this project, I hope someone can see the pattern well enough to narrow down the trigger.

  9. Flarna
    closed this as completedin 1c36eefon Jun 5, 2021

  10. Qard commented on Jun 6, 2021

    @Qard
    Qard
    on Jun 6, 2021
    Member

    This should be fixed by #38912 which will be in the next release, likely 16.3.1 or 16.4.0. If you have any further issues, let me know!

  11. targos
    added a commit that references this issue on Jun 11, 2021

    async_hooks: switch between native and context hooks correctly

    03e75fd
  12. Qard
    added a commit that references this issue on Aug 1, 2021

    async_hooks: switch between native and context hooks correctly

    be12749
  13. targos
    added a commit that references this issue on Aug 3, 2021

    async_hooks: switch between native and context hooks correctly

    b7b80e6
  14. BethGriggs
    added a commit that references this issue on Aug 12, 2021

    async_hooks: switch between native and context hooks correctly

    cbb1eaa
  15. MylesBorins
    added a commit that references this issue on Aug 31, 2021

    async_hooks: switch between native and context hooks correctly

    691c00c
  16. MayaLekova
    added a commit that references this issue on May 22, 2026

    async_hooks: switch between native and context hooks correctly

    1c36eef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    async_hooksIssues and PRs related to the async hooks subsystem.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions