Only write config file during postinstall if process env variables are present (microsoft#91793)

Rachel Macfarlane · web-flow · commit b2b488fa438e · 2020-03-02T12:45:33.000-08:00
diff --git a/extensions/github-authentication/build/postinstall.js b/extensions/github-authentication/build/postinstall.js
@@ -20,7 +20,9 @@ function main() {
 		}
 	}
 
-	fs.writeFileSync(path.join(__dirname, '../src/common/config.json'), JSON.stringify(content));
+	if (Object.keys(content).length > 0) {
+		fs.writeFileSync(path.join(__dirname, '../src/common/config.json'), JSON.stringify(content));
+	}
 }
 
 main();

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,9 @@ function main() {`
`20`	`20`	`}`
`21`	`21`	`}`
`22`	`22`
`23`		`- fs.writeFileSync(path.join(__dirname, '../src/common/config.json'), JSON.stringify(content));`
	`23`	`+ if (Object.keys(content).length > 0) {`
	`24`	`+ fs.writeFileSync(path.join(__dirname, '../src/common/config.json'), JSON.stringify(content));`
	`25`	`+ }`
`24`	`26`	`}`
`25`	`27`
`26`	`28`	`main();`