Fix exec call to remove cache folder · victorb/trymodule#15 | Good First Issue

(0 留言) (0 反應) (0 負責人)JavaScript (31 fork)github user discovery

enhancementhelp wanted

倉庫指標

Star: (1,133 star)
PR 合併指標: (PR 指標待抓取)

描述

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

貢獻者指南

研究方向: 將 exec 調用替換為 execFile 或使用 rimraf 來安全地刪除緩存文件夾。
技術棧: javascript
領域: cli
議題類型: 錯誤
難度: 2
預計時間: 1-3 小時
活動狀態: 活躍
清晰度: 清晰
前置要求: Node.jsGit
新手友善度: 75