sindresorhus/execa

Ability to kill all descendents of the child process

Open

#96 ouverte le 23 juin 2017

Voir sur GitHub
 (14 commentaires) (13 réactions) (0 assignés)JavaScript (6 091 stars) (219 forks)batch import
enhancementhelp wanted

Description

When I'm working with child processes, I sometimes end up with detached processes running indefinitely after I've killed the direct child, which can lead to blocked ports and memory leaks.

I sometimes fix this by using a module like ps-tree to find out all the descendants PIDs, and then send all of them a kill command, but it feels messy. It would be nice if execa could provide a clean, cross-platform API for this.

Maybe something like this:

const execa = require('execa');

const child = execa('./something');

// ...

child.killTree().then(() => {
  console.log('all dead!');
});

Guide contributeur

Stack technique
javascript
Domaine
backendcli
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.js child process basicsexeca API usage
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Investigate how to implement a cross platform method to kill all descendant processes. Consider using process group or tree kill techniques. Look at existing comments in the issue for prior discussion and references to ps tree module. Review Node.js documentation on child processes and signal handling.