GeezerVison is showing github.com — test: fix defect path traversal tests · nodejs/node@ae905a8
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: test/fixtures/permission/fs-traversal.js +3-3Lines changed: 3 additions & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -12,9 +12,9 @@ path.resolve = (s) => s; |
12 | 12 |
|
13 | 13 | const blockedFolder = process.env.BLOCKEDFOLDER; |
14 | 14 | const allowedFolder = process.env.ALLOWEDFOLDER; |
15 | | -const traversalPath = allowedFolder + '../file.md'; |
16 | | -const traversalFolderPath = allowedFolder + '../folder'; |
17 | | -const bufferTraversalPath = Buffer.from(allowedFolder + '../file.md'); |
| 15 | +const traversalPath = allowedFolder + '/../file.md'; |
| 16 | +const traversalFolderPath = allowedFolder + '/../folder'; |
| 17 | +const bufferTraversalPath = Buffer.from(traversalPath); |
18 | 18 | const uint8ArrayTraversalPath = new TextEncoder().encode(traversalPath); |
19 | 19 |
|
20 | 20 | { |
|
Collapse file
Copy file name to clipboardExpand all lines: test/parallel/test-permission-fs-traversal-path.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -18,7 +18,7 @@ const tmpdir = require('../common/tmpdir'); |
18 | 18 |
|
19 | 19 | const file = fixtures.path('permission', 'fs-traversal.js'); |
20 | 20 | const blockedFolder = tmpdir.path; |
21 | | -const allowedFolder = tmpdir.resolve('subdirectory/'); |
| 21 | +const allowedFolder = tmpdir.resolve('subdirectory'); |
22 | 22 | const commonPathWildcard = path.join(__filename, '../../common*'); |
23 | 23 |
|
24 | 24 | { |
|