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

fix: ignore TIMERWRAP in AsyncHooksContextManager #1530

Merged
merged 3 commits into from
Sep 21, 2020
Merged

fix: ignore TIMERWRAP in AsyncHooksContextManager #1530

merged 3 commits into from
Sep 21, 2020

Commits on Sep 14, 2020

  1. fix: ignore TIMERWRAP in AsyncHooksContextManager 

    TIMERWRAP is used internally in node.js up to version 11 to combine
timeouts with the same timeout value. This combination can lead to
false context propagation.

Every timer additionally creates a Timeout resource therefore TIMERWRAP
is not needed for correct propagation and it's safe to ignore it in
context manager.
    @Flarna
    Flarna committed Sep 14, 2020
    Configuration menu
    Copy the full SHA
    316daca View commit details
    Browse the repository at this point in the history

Commits on Sep 15, 2020

  1. Merge branch 'master' into ignore-timerwrap

    @Flarna
    Flarna committed Sep 15, 2020
    Configuration menu
    Copy the full SHA
    0242a30 View commit details
    Browse the repository at this point in the history

Commits on Sep 21, 2020

  1. Merge branch 'master' into ignore-timerwrap

    @dyladan
    dyladan committed Sep 21, 2020
    Configuration menu
    Copy the full SHA
    e8f7c9d View commit details
    Browse the repository at this point in the history