Skip to content

Fix #451: Support Spring Boot 4.0.x#452

Open
romanstrobl wants to merge 19 commits into
developfrom
issues/451-spring-boot-4
Open

Fix #451: Support Spring Boot 4.0.x#452
romanstrobl wants to merge 19 commits into
developfrom
issues/451-spring-boot-4

Conversation

@romanstrobl
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates this multi-module library to align with the Spring Boot 4.x ecosystem by adjusting dependency versions and adapting Jackson/WebFlux integration to the newer stack.

Changes:

  • Bump parent-managed Spring Boot version to 4.0.6.
  • Migrate multiple modules/tests from com.fasterxml.jackson.* to tools.jackson.* APIs and adjust WebClient codec configuration accordingly.
  • Update REST client request header handling and a few test assertions for Spring 4/Spring Framework updates.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
pom.xml Bumps the managed Spring Boot dependency version to 4.0.6.
rest-model-base/src/main/java/com/wultra/core/rest/model/base/response/ObjectResponse.java Adds JSON property ordering annotation to stabilize serialized field order.
rest-model-base/pom.xml Adds a Jackson dependency to support new JSON annotations in the model module.
rest-client-base/src/main/java/com/wultra/core/rest/client/base/DefaultRestClient.java Migrates Jackson/WebClient codec configuration and type handling for Spring Boot 4 compatibility.
rest-client-base/src/main/java/com/wultra/core/rest/client/base/RestClientConfiguration.java Switches Jackson feature types to the relocated Jackson API surface.
rest-client-base/src/test/java/com/wultra/core/rest/client/base/DefaultRestClientTest.java Updates tests to new Jackson exception types and header handling style.
rest-client-base/pom.xml Adds relocated Jackson dependency and epoll transport classes required by Netty option usage.
http-common/src/test/java/com/wultra/core/http/common/headers/UserAgentTest.java Migrates test JSON parsing to the relocated Jackson API surface.
http-common/pom.xml Updates test Jackson dependency coordinates to relocated groupId.
audit-base/src/main/java/com/wultra/core/audit/base/util/JsonUtil.java Migrates JSON serialization utilities to relocated Jackson APIs and a builder-based mapper.
audit-base/pom.xml Updates Jackson dependency coordinates and removes the JavaTime datatype module dependency.
Comments suppressed due to low confidence (1)

rest-client-base/src/main/java/com/wultra/core/rest/client/base/DefaultRestClient.java:110

  • DefaultRestClient constructor signature changed from accepting Jackson Module to JacksonModule (tools.jackson.*). This is a breaking API change for library consumers passing standard Jackson modules; consider keeping a compatibility overload or clearly versioning/documenting the breaking change as part of the Spring Boot 4 migration.
     * @param modules jackson modules
     * @throws RestClientException Thrown in case client initialization fails.
     */
    public DefaultRestClient(final RestClientConfiguration config, final JacksonModule... modules) throws RestClientException {
        // Use WebClient configuration from the config constructor parameter
        this.config = config;
        this.modules = modules == null ? Collections.emptyList() : Arrays.asList(modules);
        initializeWebClient();

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread rest-model-base/pom.xml Outdated
Copy link
Copy Markdown
Member

@banterCZ banterCZ left a comment

Choose a reason for hiding this comment

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

May we please start writing Changelog.md as we have discussed? Just one line, that we are migrating to Spring Boot 4 and Jackson 3.

Comment thread rest-model-base/pom.xml Outdated
Comment thread rest-model-base/pom.xml Outdated
Copy link
Copy Markdown
Member

@michal-rozehnal-w michal-rozehnal-w left a comment

Choose a reason for hiding this comment

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

Looks good

@romanstrobl romanstrobl requested a review from banterCZ May 13, 2026 09:17
Copy link
Copy Markdown
Member

@banterCZ banterCZ left a comment

Choose a reason for hiding this comment

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

Thanks for reflecting the comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants