Fix for stripping out TRIGGER_ env vars

matt-aitken · matt-aitken · commit 0847bee5b817 · 2024-07-15T17:11:15.000+01:00
diff --git a/apps/webapp/app/v3/environmentVariables/environmentVariablesRepository.server.ts b/apps/webapp/app/v3/environmentVariables/environmentVariablesRepository.server.ts
@@ -97,7 +97,7 @@ export class EnvironmentVariablesRepository implements Repository {
     );
 
     //get rid of empty variables
-    values = options.variables.filter((v) => v.key.trim() !== "" && v.value.trim() !== "");
+    values = values.filter((v) => v.key.trim() !== "" && v.value.trim() !== "");
     if (values.length === 0) {
       return { success: false as const, error: `You must set at least one value` };
     }

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ export class EnvironmentVariablesRepository implements Repository {`
`97`	`97`	`);`
`98`	`98`
`99`	`99`	`//get rid of empty variables`
`100`		`- values = options.variables.filter((v) => v.key.trim() !== "" && v.value.trim() !== "");`
	`100`	`+ values = values.filter((v) => v.key.trim() !== "" && v.value.trim() !== "");`
`101`	`101`	`if (values.length === 0) {`
`102`	`102`	return { success: false as const, error: `You must set at least one value` };
`103`	`103`	`}`