Skip to content

Commit 14d03ce

Browse files
fix(bench): remove unnecessary shell: true from execFileSync (#168)
execFileSync already receives args as an array, so shell: true is redundant and a minor security anti-pattern. Addresses Greptile review. Impact: 1 functions changed, 0 affected Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
1 parent eb52074 commit 14d03ce

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

scripts/lib/bench-config.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ export async function resolveBenchmarkSource() {
7070
cwd: tmpDir,
7171
stdio: 'pipe',
7272
timeout: 120_000,
73-
shell: true,
7473
});
7574
break;
7675
} catch (err) {

0 commit comments

Comments
 (0)