From ed9e0a6575e370483c2ee00d84e467602fad7c11 Mon Sep 17 00:00:00 2001
From: Monica Gupta <scorpio90m@gmail.com>
Date: Mon, 13 Sep 2021 10:23:07 -0700
Subject: [PATCH] Change few default settings for adsweb (#17029)

* device auth,notebook default settings for adsweb

* Addressed comment

Co-authored-by: hhh <gfhg>
---
 extensions/azurecore/src/extension.ts | 7 +++++++
 extensions/notebook/src/extension.ts  | 6 ++++++
 2 files changed, 13 insertions(+)

diff --git a/extensions/azurecore/src/extension.ts b/extensions/azurecore/src/extension.ts
index 5103481061..a8cd482006 100644
--- a/extensions/azurecore/src/extension.ts
+++ b/extensions/azurecore/src/extension.ts
@@ -81,6 +81,13 @@ export async function activate(context: vscode.ExtensionContext): Promise<azurec
 	if (!storagePath) {
 		return undefined;
 	}
+
+	// TODO: Since Code Grant auth doesnt work in web mode, enabling Device code auth by default for web mode. We can remove this once we have that working in web mode.
+	const config = vscode.workspace.getConfiguration('accounts.azure.auth');
+	if (vscode.env.uiKind === vscode.UIKind.Web) {
+		await config.update('deviceCode', true, vscode.ConfigurationTarget.Global);
+	}
+
 	updatePiiLoggingLevel();
 
 	// Create the provider service and activate
diff --git a/extensions/notebook/src/extension.ts b/extensions/notebook/src/extension.ts
index acc24e58f0..18e8e71353 100644
--- a/extensions/notebook/src/extension.ts
+++ b/extensions/notebook/src/extension.ts
@@ -30,6 +30,12 @@ export async function activate(extensionContext: vscode.ExtensionContext): Promi
 	IconPathHelper.setExtensionContext(extensionContext);
 
 	const appContext = new AppContext(extensionContext);
+
+	// TODO: Notebook doesn't work without root setting enabled in web mode. Once we start using non-root containers, we can remove this code.
+	const config = vscode.workspace.getConfiguration('notebook');
+	if (vscode.env.uiKind === vscode.UIKind.Web) {
+		await config.update('allowRoot', true, vscode.ConfigurationTarget.Global);
+	}
 	/**
 	 *  									***** IMPORTANT *****
 	 * If changes are made to bookTreeView.openBook, please ensure backwards compatibility with its current state.