Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit d012dd3

Browse filesBrowse files
RafaelGSSmarco-ippolito
authored andcommitted
lib: remove path.resolve from permissions.js
PR-URL: #53729 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Minwoo Jung <nodecorelab@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
1 parent 832328e commit d012dd3
Copy full SHA for d012dd3

1 file changed

-5Lines changed: 0 additions & 5 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎lib/internal/process/permission.js‎

Copy file name to clipboardExpand all lines: lib/internal/process/permission.js
-5Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,10 @@
22

33
const {
44
ObjectFreeze,
5-
StringPrototypeStartsWith,
65
} = primordials;
76

87
const permission = internalBinding('permission');
98
const { validateString } = require('internal/validators');
10-
const { resolve } = require('path');
119

1210
let experimentalPermission;
1311

@@ -25,9 +23,6 @@ module.exports = ObjectFreeze({
2523
if (reference != null) {
2624
// TODO: add support for WHATWG URLs and Uint8Arrays.
2725
validateString(reference, 'reference');
28-
if (StringPrototypeStartsWith(scope, 'fs')) {
29-
reference = resolve(reference);
30-
}
3126
}
3227

3328
return permission.has(scope, reference);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.