Cleanup on SIGINT & uncaughtException

[ajmas]

Would it be worth supporting cleanup on non-normal process exit, such that:

/** Removes the PID file on normal process exit. */
Pid.prototype.removeOnExit = function() {
    // do something when app is closing
    process.on('exit', this.remove.bind(this));
    // catches ctrl+c event
    process.on('SIGINT', this.remove.bind(this));
    // catches uncaught exceptions
    process.on('uncaughtException', this.remove.bind(this));
};

When doing development I find that the process file remains when I hit ctrl-c.

[MathieuTurcotte]

This is by design since setting handlers for these events override the default node behavior which is to print a stack trace and exit the process. I didn't want this library to interfere with the default behavior since it would be surprising. The way around it is to specify handlers for uncaught exceptions and signals that call process.exit or remove the pid file explicitly before exiting. I'd be open to suggestions in order to integrate some scaffolding for people who want the behavior you describe by default, but it seems unnecessary.

[Alexis-benoist]

Hey are you ready to accept PR to make this optional?