Fix AzureOperationException: get details of all cached subscriptions (#…

…472)
JetBrains · Apr 12, 2021 · 9503d56 · 9503d56
1 parent 99dfe83
commit 9503d56
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/Utils/azuretools-core/src/com/microsoft/azuretools/sdkmanage/AzureManagerBase.java b/Utils/azuretools-core/src/com/microsoft/azuretools/sdkmanage/AzureManagerBase.java
@@ -16,6 +16,7 @@
 import com.microsoft.azure.management.resources.Subscription;
 import com.microsoft.azure.management.resources.Tenant;
 import com.microsoft.azure.toolkit.lib.common.operation.AzureOperation;
+import com.microsoft.azure.toolkit.lib.common.operation.AzureOperationException;
 import com.microsoft.azure.toolkit.lib.common.rest.RestExceptionHandlerInterceptor;
 import com.microsoft.azuretools.adauth.AuthException;
 import com.microsoft.azuretools.authmanage.*;
@@ -137,6 +138,13 @@ public List<Pair<Subscription, Tenant>> getSubscriptionsWithTenant() {
                     Throwables.getCausalChain(e).stream().anyMatch(tenantError)) {
                     // TODO: @wangmi better to notify user
                     LOGGER.log(Level.WARNING, e.getMessage(), e);
+                } else if (e instanceof AzureOperationException) {
+                    // TODO: @maartenba/@sdubov
+                    // This is where upstream issue happens:
+                    // https://github.com/microsoft/azure-tools-for-java/issues/5053
+                    // https://github.com/microsoft/azure-tools-for-java/issues/4977
+                    // Next merge, see if upstream has been updated.
+                    LOGGER.log(Level.SEVERE, e.getMessage(), e);
                 } else {
                     throw e;
                 }