victorb/trymodule

Fix exec call to remove cache folder

Aperta

#15 aperta il 29 mar 2016

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (31 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (1133 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

https://github.com/VictorBjelkholm/trymodule/blob/2030806d790213d426456959fdc7a69773a4c6f7/index.js#L59

Oh noes. Please don't. Never do string concat for commands, use execFile instead. Never pass environment variables directly to rm without sanitation, better yet, never spawn rm. Use a JS tools, like rimraf or even better del.

From: https://github.com/sindresorhus/awesome-nodejs/pull/477#event-606502245

Guida contributor

Direzione di ricerca
Sostituisci la chiamata exec con execFile o usa rimraf per rimuovere in sicurezza la cartella cache.
Tech stack
javascript
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75