Skip to content

Commit

Permalink
lib: remove path.resolve from permissions.js
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
RafaelGSS authored and marco-ippolito committed Aug 19, 2024
1 parent 832328e commit d012dd3
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions lib/internal/process/permission.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,10 @@

const {
ObjectFreeze,
StringPrototypeStartsWith,
} = primordials;

const permission = internalBinding('permission');
const { validateString } = require('internal/validators');
const { resolve } = require('path');

let experimentalPermission;

Expand All @@ -25,9 +23,6 @@ module.exports = ObjectFreeze({
if (reference != null) {
// TODO: add support for WHATWG URLs and Uint8Arrays.
validateString(reference, 'reference');
if (StringPrototypeStartsWith(scope, 'fs')) {
reference = resolve(reference);
}
}

return permission.has(scope, reference);
Expand Down

0 comments on commit d012dd3

Please sign in to comment.