Disable welcome page when user is root

Fixes #3068
microsoft · Apr 7, 2016 · ad7e861 · ad7e861
1 parent 6456f85
commit ad7e861
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/vs/workbench/parts/gettingStarted/common/abstractGettingStarted.ts b/src/vs/workbench/parts/gettingStarted/common/abstractGettingStarted.ts
@@ -34,8 +34,11 @@ export abstract class AbstractGettingStarted implements IWorkbenchContribution {
 
 	protected handleWelcome(): void {
 		let firstStartup = !this.storageService.get(AbstractGettingStarted.hideWelcomeSettingskey);
+		let isRootUser = process.getuid() === 0;
 
-		if (firstStartup && this.welcomePageURL) {
+		// Don't open the welcome page as the root user, This is due to a bug with xdg-open which
+		// recommends against running itself as root.
+		if (firstStartup && this.welcomePageURL && !isRootUser) {
 			this.telemetryService.getTelemetryInfo().then(info=>{
 				let url = this.getUrl(info);
 				this.openExternal(url);