Remove deprecated azure graph endpoints (#24460)

* remove deprecated azure graph endpoints

* cleanup enums

* cleanup ProviderSettingsJson: remove graphResource

* change enums back

extensions/azurecore/src/account-provider/auths/azureAuth.ts

@@ -68,8 +68,7 @@ export abstract class AzureAuth implements vscode.Disposable {
 		this.redirectUri = this.metadata.settings.redirectUri;
 		this.clientId = this.metadata.settings.clientId;
 		this.resources = [
-			this.metadata.settings.armResource,
+			this.metadata.settings.armResource
-			this.metadata.settings.graphResource,
 		];
 		if (this.metadata.settings.sqlResource) {
 			this.resources.push(this.metadata.settings.sqlResource);

extensions/azurecore/src/account-provider/interfaces.ts

@@ -7,7 +7,6 @@ import * as azurecore from 'azurecore';
 
 export const enum SettingIds {
 	marm = 'marm',
-	graph = 'graph',
 	msgraph = 'msgraph',
 	arm = 'arm',
 	sql = 'sql',
@@ -49,7 +48,6 @@ export type ProviderSettingsJson = {
 				host: string,
 				clientId: string,
 				microsoftResource: string,
-				graphResource: string,
 				msGraphResource?: string,
 				armResource: string,
 				sqlResource: string,

extensions/azurecore/src/account-provider/providerSettings.ts

@@ -23,11 +23,6 @@ const publicAzureSettings: ProviderSettings = {
 				endpoint: 'https://management.core.windows.net/',
 				azureResourceId: AzureResource.MicrosoftResourceManagement
 			},
-			graphResource: {
-				id: SettingIds.graph,
-				endpoint: 'https://graph.windows.net/',
-				azureResourceId: AzureResource.Graph
-			},
 			msGraphResource: {
 				id: SettingIds.msgraph,
 				endpoint: 'https://graph.microsoft.com/',
@@ -103,11 +98,6 @@ const usGovAzureSettings: ProviderSettings = {
 				endpoint: 'https://management.core.usgovcloudapi.net/',
 				azureResourceId: AzureResource.MicrosoftResourceManagement
 			},
-			graphResource: {
-				id: SettingIds.graph,
-				endpoint: 'https://graph.windows.net/',
-				azureResourceId: AzureResource.Graph
-			},
 			msGraphResource: {
 				id: SettingIds.msgraph,
 				endpoint: 'https://graph.microsoft.us/',
@@ -177,11 +167,6 @@ const chinaAzureSettings: ProviderSettings = {
 				endpoint: 'https://management.core.chinacloudapi.cn/',
 				azureResourceId: AzureResource.MicrosoftResourceManagement
 			},
-			graphResource: {
-				id: SettingIds.graph,
-				endpoint: 'https://graph.chinacloudapi.cn',
-				azureResourceId: AzureResource.Graph
-			},
 			msGraphResource: {
 				id: SettingIds.msgraph,
 				endpoint: 'https://microsoftgraph.chinacloudapi.cn',

extensions/azurecore/src/azurecore.d.ts

@@ -93,11 +93,6 @@ declare module 'azurecore' {
 		 */
 		microsoftResource: Resource
 
-		/**
-		 * Information that describes the AAD graph resource
-		 */
-		graphResource: Resource;
-
 		/**
 		 * Information that describes the MS graph resource
 		 */

extensions/azurecore/src/utils.ts

@@ -217,11 +217,6 @@ function buildCustomCloudProviderSettings(customProvider: ProviderSettingsJson):
 					endpoint: customProvider.settings.metadata.endpoints.armResource,
 					azureResourceId: AzureResource.ResourceManagement
 				},
-				graphResource: {
-					id: SettingIds.graph,
-					endpoint: customProvider.settings.metadata.endpoints.graphResource,
-					azureResourceId: AzureResource.Graph
-				},
 				azureStorageResource: {
 					id: SettingIds.storage,
 					endpoint: customProvider.settings.metadata.endpoints.azureStorageResource.endpoint,

src/sql/azdata.d.ts

@@ -2524,10 +2524,7 @@ declare module 'azdata' {
 		 * Azure Key Vault
 		 */
 		AzureKeyVault = 3,
-		/**
+		// 4 (formerly Azure Graph) is no longer used.
-		 * Azure AD Graph
-		 */
-		Graph = 4,
 		/**
 		 * Microsoft Resource Management
 		 */

src/sql/workbench/api/common/sqlExtHostTypes.ts

@@ -476,7 +476,7 @@ export enum AzureResource {
 	Sql = 1,
 	OssRdbms = 2,
 	AzureKeyVault = 3,
-	Graph = 4,
+	// 4 (formerly Azure Graph) is no longer used.
 	MicrosoftResourceManagement = 5,
 	AzureDevOps = 6,
 	MsGraph = 7,