browsh-org/browsh

Kill the headless Firefox process by PID, not by grepping `ps`

Open

#61 ouverte le 9 juil. 2018

Voir sur GitHub
 (2 commentaires) (3 réactions) (0 assignés)JavaScript (455 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (15 996 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

This was first brought up by a user that had an existing Selenium session: #59

If Browsh's Firefox process can have an identifying mark to it, then we can detect that instead of a crude detection of any headless Firefox process.

Guide contributeur

Direction de recherche
Examinez comment Browsh lance actuellement Firefox, probablement dans le code du lanceur du navigateur. Comprenez la méthode actuelle de terminaison via grep ps et l'issue référencée (#59). Implémentez un identifiant unique (par exemple, un argument de ligne de commande personnalisé comme ' browsh pid') à passer à Firefox, afin que le processus puisse être tué directement par PID. Assurez vous que l'identifiant soit stocké et récupérable après le lancement.
Stack technique
javascript
Domaine
cli
Type d'issue
Fonctionnalité
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Basic Node.jsUnderstanding of child processesFamiliarity with Firefox CLI flags
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65