Prepare for Spring Cloud Azure5.1.0 (#34706)

* update sdk-bom to 1.2.12 * App Config Key Vault Cred Fix - Spring 3 (#34695) --------- Co-authored-by: Matt Metcalf <[email protected]>
Azure · Apr 28, 2023 · 0825d9f · 0825d9f
1 parent b324e6b
commit 0825d9f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/sdk/boms/spring-cloud-azure-dependencies/pom.xml b/sdk/boms/spring-cloud-azure-dependencies/pom.xml
@@ -51,7 +51,7 @@
       <dependency>
         <groupId>com.azure</groupId>
         <artifactId>azure-sdk-bom</artifactId>
-        <version>1.2.11</version>   <!-- NOTE: This should be updated manually. -->
+        <version>1.2.12</version>   <!-- NOTE: This should be updated manually. -->
         <type>pom</type>
         <scope>import</scope>
       </dependency>

diff --git a/sdk/spring/CHANGELOG.md b/sdk/spring/CHANGELOG.md
@@ -7,7 +7,7 @@
 ### Spring Cloud Azure Dependencies (BOM)
 
 #### Dependency Updates
-- Upgrade `azure-sdk-bom` to 1.2.11.
+- Upgrade `azure-sdk-bom` to 1.2.12.
 - Upgrade `azure-resourcemanager` to 2.25.0.
 
 #### Features Added
@@ -53,6 +53,12 @@ This section includes changes in `spring-cloud-azure-core`, `spring-cloud-azure-
 #### Breaking Changes
 - Deprecated `CloudType.AZURE_GERMANY` and remove `AzureEnvironmentProperties.AZURE_GERMANY` [#34663](https://github.com/Azure/azure-sdk-for-java/pull/34663).
 
+### Spring Cloud Azure Appconfiguration Config
+This section includes changes in `spring-cloud-azure-appconfiguration-config` and `spring-cloud-azure-appconfiguration-config-web` modules.
+
+#### Bugs Fixed
+- Fixes issue where credential from Azure Spring global properties was being overridden [#34695](https://github.com/Azure/azure-sdk-for-java/pull/34695).
+
 ## 5.0.0 (2023-01-17)
 - This release is compatible with Spring Boot 3.0.0-3.0.1. (Note: 3.0.x (x>1) should be supported, but they aren't tested with this release.)
 - This release is compatible with Spring Cloud 2022.0.0. (Note: 2022.0.x (x>0) should be supported, but they aren't tested with this release.)

diff --git a/...ud/appconfiguration/config/implementation/stores/AppConfigurationSecretClientManager.java b/...ud/appconfiguration/config/implementation/stores/AppConfigurationSecretClientManager.java
@@ -52,7 +52,7 @@ public AppConfigurationSecretClientManager(String endpoint, SecretClientCustomiz
     AppConfigurationSecretClientManager build() {
         SecretClientBuilder builder = secretClientFactory.build();
 
-        if (credentialConfigured) {
+        if (!credentialConfigured) {
             // System Assigned Identity.
             builder.credential(new ManagedIdentityCredentialBuilder().build());
         }