Skip to content

Commit 5450e48

Browse files
BridgeARZYSzys
authored andcommitted
worker: simplify filename checks
Replace a couple of checks with a single regular expression. PR-URL: #27233 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
1 parent c61c722 commit 5450e48

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

lib/internal/worker.js

+1-5
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,7 @@ class Worker extends EventEmitter {
6868
options.execArgv);
6969
}
7070
if (!options.eval) {
71-
if (!path.isAbsolute(filename) &&
72-
!filename.startsWith('./') &&
73-
!filename.startsWith('../') &&
74-
!filename.startsWith('.' + path.sep) &&
75-
!filename.startsWith('..' + path.sep)) {
71+
if (!path.isAbsolute(filename) && !/^\.\.?[\\/]/.test(filename)) {
7672
throw new ERR_WORKER_PATH(filename);
7773
}
7874
filename = path.resolve(filename);

0 commit comments

Comments
 (0)