fix env split

devksingh4 · devksingh4 · commit 7db83cc165f5 · 2023-01-22T13:06:27.000-06:00
diff --git a/config.js b/config.js
@@ -10,7 +10,7 @@ function makeSecret(length) {
   return result;
 }
 
-exports.groups_permitted = process.env.groupsPermitted.split(',') || ['ACM Link Shortener Managers', 'ACM Exec', 'ACM Infra Leadership'];
+exports.groups_permitted = process.env.groupsPermitted ? process.env.groupsPermitted.split(',') : ['ACM Link Shortener Managers', 'ACM Exec', 'ACM Infra Leadership'];
 
 exports.branding = {
   title: process.env.brandTitle || "ACM Link Shortener",

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ function makeSecret(length) {`
`10`	`10`	`return result;`
`11`	`11`	`}`
`12`	`12`
`13`		`-exports.groups_permitted = process.env.groupsPermitted.split(',') \|\| ['ACM Link Shortener Managers', 'ACM Exec', 'ACM Infra Leadership'];`
	`13`	`+exports.groups_permitted = process.env.groupsPermitted ? process.env.groupsPermitted.split(',') : ['ACM Link Shortener Managers', 'ACM Exec', 'ACM Infra Leadership'];`
`14`	`14`
`15`	`15`	`exports.branding = {`
`16`	`16`	`title: process.env.brandTitle \|\| "ACM Link Shortener",`