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

lib: rearm pre-existing signal event registrations #24651

Closed
wants to merge 1 commit into from
Closed

lib: rearm pre-existing signal event registrations #24651

wants to merge 1 commit into from

Commits on Nov 26, 2018

  1. lib: rearm pre-existing signal event registrations 

    process.on('somesignal', ...) semantics expect the process to catch the
signal and invoke the associated handler. `setupSignalHandlers` perform
the additional task of preparing the libuv signal handler and associate
it with the event handler. It is possible that by the time this is
setup there could be pre-existing registrations that pre-date this setup
in the boot sequence.

So rearm pre-existing signal event registrations to get upto speed.

Ref: nodejs#22712 (comment)
    @gireeshpunathil
    gireeshpunathil committed Nov 26, 2018
    Configuration menu
    Copy the full SHA
    5f0cbdb View commit details
    Browse the repository at this point in the history