From ca8bc7ece8a37bc49aa3668cb64eada3027305d5 Mon Sep 17 00:00:00 2001 From: Asher Gomez Date: Wed, 21 Feb 2024 08:40:32 +1100 Subject: fix(ext/node): permission prompt for missing `process.env` permissions (#22487) Closes #18665 Closes #20213 --- ext/node/polyfills/_process/process.ts | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) (limited to 'ext/node/polyfills') diff --git a/ext/node/polyfills/_process/process.ts b/ext/node/polyfills/_process/process.ts index 4fae1bc1b..cc52e7706 100644 --- a/ext/node/polyfills/_process/process.ts +++ b/ext/node/polyfills/_process/process.ts @@ -36,17 +36,10 @@ export const nextTick = _nextTick; /** Wrapper of Deno.env.get, which doesn't throw type error when * the env name has "=" or "\0" in it. */ function denoEnvGet(name: string) { - const perm = - Deno.permissions.querySync?.({ name: "env", variable: name }).state ?? - "granted"; // for Deno Deploy - // Returns undefined if the env permission is unavailable - if (perm !== "granted") { - return undefined; - } try { return Deno.env.get(name); } catch (e) { - if (e instanceof TypeError) { + if (e instanceof TypeError || e instanceof Deno.errors.PermissionDenied) { return undefined; } throw e; -- cgit v1.2.3