Merge pull request #955 from edify42/master

k8s-ci-robot · web-flow · commit 41b79162d477 · 2023-01-05T09:13:59.000-08:00
Copy the nodejs process.env rather than mutate the global state
diff --git a/src/exec_auth.ts b/src/exec_auth.ts
@@ -97,7 +97,9 @@ export class ExecAuth implements Authenticator {
         }
         let opts = {};
         if (exec.env) {
-            const env = process.env;
+            const env = {
+                ...process.env,
+            };
             exec.env.forEach((elt) => (env[elt.name] = elt.value));
             opts = { ...opts, env };
         }

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,9 @@ export class ExecAuth implements Authenticator {`
`97`	`97`	`}`
`98`	`98`	`let opts = {};`
`99`	`99`	`if (exec.env) {`
`100`		`- const env = process.env;`
	`100`	`+ const env = {`
	`101`	`+ ...process.env,`
	`102`	`+ };`
`101`	`103`	`exec.env.forEach((elt) => (env[elt.name] = elt.value));`
`102`	`104`	`opts = { ...opts, env };`
`103`	`105`	`}`