Removed localization from error messages (#21085)

* removed localization from error messages

* use logger

* localize message displayed to user
This commit is contained in:
Christopher Suh
2022-11-02 18:17:59 -07:00
committed by GitHub
parent dc5f7854a2
commit e87995527b

View File

@@ -11,6 +11,7 @@ import { IAzureResourceSubscriptionService } from '../interfaces';
import { TokenCredentials } from '@azure/ms-rest-js';
import { AzureSubscriptionError } from '../errors';
import { AzureResourceErrorMessageUtil } from '../utils';
import { Logger } from '../../utils/Logger';
import * as nls from 'vscode-nls';
const localize = nls.loadMessageBundle();
@@ -45,12 +46,12 @@ export class AzureResourceSubscriptionService implements IAzureResourceSubscript
}
else if (!account.isStale) {
const errorMsg = localize('azure.resource.tenantTokenError', "Failed to acquire Access Token for account '{0}' (tenant '{1}').", account.displayInfo.displayName, tenantId);
console.warn(errorMsg);
Logger.error(`Failed to acquire Access Token for account '${account.displayInfo.displayName}' (tenant '${tenantId}').`);
void vscode.window.showWarningMessage(errorMsg);
}
} catch (error) {
const errorMsg = localize('azure.resource.tenantSubscriptionsError', "Failed to get subscriptions for account {0} (tenant '{1}'). {2}", account.displayInfo.displayName, tenantId, AzureResourceErrorMessageUtil.getErrorMessage(error));
console.warn(errorMsg);
Logger.error(`Failed to get subscriptions for account ${account.displayInfo.displayName} (tenant '${tenantId}'). ${AzureResourceErrorMessageUtil.getErrorMessage(error)}`);
errors.push(error);
void vscode.window.showWarningMessage(errorMsg);
}