Skip to content

Spawn children in their own process groups, and deliver SIGINT to each group #177

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

Open
benweint wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Spawn children in their own process groups, and deliver SIGINT to each group #177

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
68a2921
Create process group for each child + subprocesses (fixes #176)
benweint Dec 4, 2023
c023d04
Handle delivery of multiple SIGINT signals without exiting
benweint Dec 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Handle delivery of multiple SIGINT signals without exiting 
Signed-off-by: Ben Weintraub <ben.weintraub@nytimes.com>
  • Loading branch information
@benweint
benweint committed Dec 17, 2023
commit c023d04802e489b483fdbf8b39de1ba42067489d
6 changes: 5 additions & 1 deletion nf.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,12 @@ var calculatePadding = _requirements.calculatePadding;
var startProxies = require('./lib/proxy').startProxies;
var startForward = require('./lib/forward').startForward;

var interrupted = false;

// Kill All Child Processes on SIGINT
process.once('SIGINT', function() {
process.on('SIGINT', function() {
if (interrupted) return;
interrupted = true;
display.Warn('Interrupted by User');
emitter.emit('killall', 'SIGINT');
});
Expand Down