browsh-org/browsh

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

Open

#61 aperta il 9 lug 2018

Vedi su GitHub
 (2 commenti) (3 reazioni) (0 assegnatari)JavaScript (455 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (15.996 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina come Browsh attualmente avvia Firefox, probabilmente nel codice del launcher del browser. Comprendi il metodo attuale di terminazione tramite grep ps e l'issue di riferimento (#59). Implementa un identificatore univoco (ad esempio, un argomento da riga di comando personalizzato come ' browsh pid') da passare a Firefox, in modo che il processo possa essere terminato direttamente tramite PID. Assicurati che l'identificatore venga memorizzato e sia recuperabile dopo la generazione.
Tech stack
javascript
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Basic Node.jsUnderstanding of child processesFamiliarity with Firefox CLI flags
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65