fix issue Azure#16595 (Azure#16673)

* Fix Azure#16595
AzureSDKAutomation · Oct 22, 2020 · 89cf240 · 89cf240
1 parent fe7f5b9
commit 89cf240
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/...main/java/com/azure/spring/cloud/autoconfigure/context/AzureContextAutoConfiguration.java b/...main/java/com/azure/spring/cloud/autoconfigure/context/AzureContextAutoConfiguration.java
@@ -17,6 +17,8 @@
 import com.azure.spring.cloud.context.core.impl.DefaultCredentialsProvider;
 import com.microsoft.rest.RestClient;
 import java.io.IOException;
+import java.util.Optional;
+
 import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
@@ -56,12 +58,10 @@ public Azure azure(AzureTokenCredentials credentials, AzureProperties azurePrope
             .withInterceptor(new ResourceManagerThrottlingInterceptor()).withUserAgent(SPRING_CLOUD_USER_AGENT)
             .build();
 
-        if (azureProperties.getSubscriptionId() == null) {
-            return Azure.authenticate(restClient, credentials.domain()).withDefaultSubscription();
-        } else {
-            return Azure.authenticate(restClient, credentials.domain())
-                .withSubscription(azureProperties.getSubscriptionId());
-        }
+        String subscriptionId = Optional.of(azureProperties.getSubscriptionId())
+                                        .orElseGet(credentials::defaultSubscriptionId);
+        return Azure.authenticate(restClient, credentials.domain())
+                    .withSubscription(subscriptionId);
     }
 
     @Bean