Test spring boot 4 #47330

Netyyyy · 2025-11-19T06:40:08Z

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

The pull request does not introduce [breaking changes]
CHANGELOG is updated for new features, bug fixes or other significant changes.
I have read the contribution guidelines.

General Guidelines and Best Practices

Title of the pull request is clear and informative.
There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

Pull request includes test coverage for the included changes.

…025.1.0-M4 Co-authored-by: Netyyyy <[email protected]>

…nonnull

…nonnull annotations Co-authored-by: Netyyyy <[email protected]>

Co-authored-by: Netyyyy <[email protected]>

…025.1.0-M4 (partial) Co-authored-by: Netyyyy <[email protected]>

rujche · 2025-11-27T08:48:50Z

eng/versioning/external_dependencies.txt

-springboot3_org.springframework.boot:spring-boot-starter;3.5.5
-springboot3_org.springframework.boot:spring-boot-test;3.5.5
-springboot3_org.springframework.boot:spring-boot-testcontainers;3.5.5
+springboot3_org.springframework.boot:spring-boot-actuator-autoconfigure;4.0.0-RC2


Change the prefix from springboot3 to springboot4.

Change Spring Boot version from 4.0.0-RC2 to 4.0.0.

rujche · 2025-11-27T08:51:17Z

eng/versioning/external_dependencies.txt

 springboot3_org.springframework.cloud:spring-cloud-bus;4.3.0
 springboot3_org.springframework.cloud:spring-cloud-context;4.3.0
-springboot3_org.springframework.cloud:spring-cloud-starter-stream-kafka;4.3.0
+springboot3_org.springframework.cloud:spring-cloud-starter-stream-kafka;5.0.0-RC1


Same here. Use release version instead of RC (Release Candidate) version.

Is it right that org.springframework.cloud:spring-cloud-starter-stream-kafka upgraded the major version but org.springframework.cloud:spring-cloud-bus didn't update the version?

rujche · 2025-11-27T08:52:02Z

sdk/spring/CHANGELOG.md

@@ -1,4 +1,6 @@
 # Release History
+# (Unreleased)
+Upgrade Spring Boot dependencies version to 4.0.0-RC2 and Spring Cloud dependencies version to 2025.1.0-M4


rujche · 2025-11-27T08:57:25Z

sdk/spring/azure-spring-data-cosmos/pom.xml

+    </dependency>
+    <dependency>
+      <groupId>org.springframework.boot</groupId>
+      <artifactId>spring-boot-persistence</artifactId>


This is only used for test. Please change to test scope.

rujche · 2025-11-27T08:59:34Z

sdk/spring/pom.xml

+        <enabled>false</enabled>
+      </snapshots>
+    </pluginRepository>
+  </pluginRepositories>


This is not necessary anymore. Refs: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-4.0-Release-Notes

rujche · 2025-11-27T09:00:41Z

sdk/spring/scripts/spring_boot_4.0.0-RC2_managed_external_dependencies.txt

Mark: Use 4.0.0 instead of 4.0.0-RC2

rujche · 2025-11-27T09:27:11Z

...d/autoconfigure/implementation/aad/security/AadJwtBearerGrantRequestParametersConverter.java

+ * This is a special JWT Bearer flow implementation for Microsoft identify platform.
+ *
+ * @since 4.3.0
+ * @see <a href="https://docs.microsoft.com/azure/active-directory/develop/v2-oauth2-on-behalf-of-flow">OAuth 2.0 On-Behalf-Of</a>


The @since 4.3.0 is not right, it should be next release version.

rujche · 2025-11-27T09:27:50Z

...d/autoconfigure/implementation/aad/security/AadJwtBearerGrantRequestParametersConverter.java

+        if (ClientAuthenticationMethod.CLIENT_SECRET_POST.equals(clientRegistration.getClientAuthenticationMethod())) {
+            parameters.add(OAuth2ParameterNames.CLIENT_ID, clientRegistration.getClientId());
+            parameters.add(OAuth2ParameterNames.CLIENT_SECRET, clientRegistration.getClientSecret());
+        }


Why above parameters are necessary since Spring Boot 4?

rujche · 2025-11-28T01:19:03Z

...d/autoconfigure/implementation/aad/security/AadJwtBearerGrantRequestParametersConverter.java

+        if (ClientAuthenticationMethod.CLIENT_SECRET_POST.equals(clientRegistration.getClientAuthenticationMethod())) {
+            parameters.add(OAuth2ParameterNames.CLIENT_ID, clientRegistration.getClientId());
+            parameters.add(OAuth2ParameterNames.CLIENT_SECRET, clientRegistration.getClientSecret());
+        }


For the parameters like scope / client-id, client-secret, I guess the default converter in spring security also need to do this. Can we reuse the code in the default converter?

rujche · 2025-11-28T01:34:35Z

...zure-core/src/main/java/com/azure/spring/integration/core/handler/DefaultMessageHandler.java

-            if (this.sendCallback != null) {
-                this.sendCallback.onFailure(ex);
-            }
+        CompletableFuture<T> future = mono.toFuture();


Why it's necessary to change from Mono to Future?

…framework.data.core.TypeInformation

…gate

Netyyyy and others added 29 commits November 12, 2025 09:55

Create versions.agent.md

c00c845

Update Spring Cloud Azure upgrade instructions

f83e7ed

Update springcloudazure-versions-upgrade.agent.md

8917931

Update springcloudazure-versions-upgrade.agent.md

6bc4875

Update springcloudazure-versions-upgrade.agent.md

a946213

Update springcloudazure-versions-upgrade.agent.md

8166d4e

Initial plan

a30c5db

chore(spring): upgrade Spring Boot to 4.0.0-RC2 and Spring Cloud to 2…

8971091

…025.1.0-M4 Co-authored-by: Netyyyy <[email protected]>

fix: suppress deprecation warnings for Spring Framework 7.0 @Nullable/@…

4039c61

…nonnull annotations Co-authored-by: Netyyyy <[email protected]>

replace ListenableFutureCallback

de5beaf

add spring-boot-persistence and replace

0991f97

suppress warnings

304dce1

fix ConfigurableBootstrapContext/EnvironmentPostProcessor path

f3b4862

suppress warning

6b69885

replace with spring-boot-kafka

d5992ab

replace alwaysApplyingWhenNonNull() with always()

ee56cc0

replace with spring-boot-data-redis

b2e5e8b

replace with spring-boot-jms

d4ba1b4

replace with

1a3185d

replace with spring-boot-jdbc

351b341

add spring-boot-restclient

7895c09

add spring-boot-security

eab4621

add spring-boot-jackson

89bd0f0

add spring-boot-http-converter

353b405

replace new OAuth2ClientAutoConfiguration

0f68f24

replace containsKey with containsHeader

180293b

fix Bootstrap

b9f0879

fix health

09fc1d6

add spring-boot-health

ec9e14e

github-actions bot added the azure-spring All azure-spring related issues label Nov 19, 2025

Netyyyy and others added 13 commits November 20, 2025 08:27

fix

e07c256

fix

663c11a

fix

f22be56

Merge branch 'main' into copilot/update-spring-boot-version-again

d2acc5f

replace spring-boot-starter-web with spring-boot-starter-webmvc

31b56e5

chore(spring): update Spring Security to 7.0.0-RC3

c3c1735

Co-authored-by: Netyyyy <[email protected]>

chore(spring): upgrade Spring Boot to 4.0.0-RC2 and Spring Cloud to 2…

004c76d

…025.1.0-M4 (partial) Co-authored-by: Netyyyy <[email protected]>

update aad/aadb2c

e61f7b6

update aad/aadb2c

2173524

fix replace ListenableFutureCallback

48df3f3

fix spring integration ut

c0b85fd

fix spring integration ut

a4e92b4

fix

339c49f

rujche reviewed Nov 27, 2025

View reviewed changes

sdk/spring/scripts/spring_boot_4.0.0-RC2_managed_external_dependencies.txt Outdated

Copy link

Member

rujche Nov 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mark: Use 4.0.0 instead of 4.0.0-RC2

rujche reviewed Nov 27, 2025

View reviewed changes

rujche reviewed Nov 28, 2025

View reviewed changes

Netyyyy added 7 commits November 28, 2025 15:52

version update

5bc36b4

replace org.springframework.data.util.TypeInformation with org.spring…

f775675

…framework.data.core.TypeInformation

replace with ParametersSource.of()

ec36cf0

replace QueryMethodEvaluationContextProvider with ValueExpressionDele…

31fc7b1

…gate

replace ClassTypeInformation.from with TypeInformation.of

20376f1

suppress warning

0366061

update comments

a39db5a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Test spring boot 4 #47330

Test spring boot 4 #47330

Uh oh!

Netyyyy commented Nov 19, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 27, 2025

Uh oh!

rujche Nov 28, 2025

Uh oh!

rujche Nov 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Test spring boot 4 #47330

Are you sure you want to change the base?

Test spring boot 4 #47330

Uh oh!

Conversation

Netyyyy commented Nov 19, 2025

Description

All SDK Contribution checklist:

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants