Feature/ab#2020 update to spring boot 4

[witchpou]

Description

• Redis Listener needed to berefactored
• Github actions are refactored

Motivation and Context

How has this been tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[Copilot]

Pull request overview

This PR attempts to upgrade the project to Spring Boot 4 and Java 25, along with refactoring the Redis listener architecture and updating various dependencies. However, this PR contains critical issues with non-existent versions that must be addressed before it can be merged.

Key Changes

• Upgrade to Spring Boot "4.0.1" and Java "25" (both versions do not exist)
• Migration from com.fasterxml..jackson to tools.jackson packages
• Redis listener refactored from individual Runner subscriptions to a centralized StreamSubscriptionService
• Test framework updates (@MockBean → @MockitoBean)
• Removal of Spring Security dependencies and annotations
• Updated GitHub Actions workflows for Java 25

Reviewed changes

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

Show a summary per file

File	Description
pom.xml	Upgrades Spring Boot to non-existent version 4.0.1, Java to 25, and updates dependency versions
persistence/pom.xml	Reorganizes dependencies, adds Jackson starter, updates Flyway dependency
service/pom.xml	Adds log4j2 starter, updates test dependencies, removes explicit lettuce/security dependencies
rest/pom.xml	Removes Spring Security and actuator dependencies, updates test dependencies
application/pom.xml	Removes Spring Security and OAuth2 dependencies
service/.../StreamSubscriptionService.java	New service that centralizes Redis stream subscription logic with scheduled resubscription
service/.../SimpleMessageListener.java	New listener implementing StreamListener to handle Redis messages and dispatch to runners
service/.../RedisConfiguration.java	Refactored to add conditional bean creation and StringRedisTemplate configuration
service/.../RunnerInterface.java	Renamed method from messageHandler to handleMessage
service/.../LineCrossingRunner.java	Removed direct Redis subscription logic, now receives messages from SimpleMessageListener
service/.../FlowRunner.java	Changed from @service to @component, removed direct Redis subscription logic
service/.../AreaOccupancyRunner.java	Removed direct Redis subscription logic
persistence/.../ZonedDateTimeSerializer.java	Migrated to tools.jackson API with combined Serializer/Deserializer classes
persistence/.../ZonedDateTimeDeserializer.java	Removed (merged into ZonedDateTimeSerializer)
persistence/.../PrimaryDataSourceConfig.java	Updated imports for relocated Spring Boot classes
persistence/.../AnalyticsDataSourceConfig.java	Updated imports for relocated Spring Boot classes
service/.../GeoJsonSenderService.java	Migrated to tools.jackson JsonMapper, updated exception handling
rest/.../AbstractControllerIntegrationTest.java	Updated to use JsonMapper and new test annotations
rest/.../AbstractControllerAcceptanceTest.java	Updated to use JsonMapper
service/.../MetadataServiceTest.java	Updated from @MockBean to @MockitoBean
Test files	Added IOException handling and resource cleanup for SaeDump
application.properties	Updated Redis configuration property names
.github/workflows/*.yml	Updated all workflows to use Java 25 and actions/setup-java@v5
rest/allowedroles/*.java	Removed IsAdmin, IsReader, IsUser security annotations
application/.../Application.java	Removed ObjectMapper bean and security exclusions

---

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

[Copilot]

The saeDump resource should be properly closed even if an exception occurs. The close call is currently after the iteration, which means if an exception occurs during the loop, the resource won't be closed. Consider using try-with-resources instead of manual close.

[witchpou]

@copilot open a new pull request to apply changes based on this feedback

[witchpou]

@copilot open a new pull request to apply changes based on this feedback

[Copilot]

The deserialize method should handle exceptions that may be thrown by jsonParser.getString(). Consider wrapping this in a try-catch block or declaring the exception in the method signature.

Suggested change

	return ZonedDateTime.parse(jsonParser.getString());
	try {
	String value = jsonParser.getString();
	return value != null ? ZonedDateTime.parse(value) : null;
	} catch (Exception e) {
	throw new RuntimeException("Failed to deserialize ZonedDateTime", e);
	}

[Copilot]

The imported exception class IOException is declared but never used in this test method. The method signature now declares throws IOException, but the actual operations don't throw this exception - only the saeDump.close() call could potentially throw it, and that's already handled by the method signature.

[Copilot]

@witchpou I've opened a new pull request, #102, to work on those changes. Once the pull request is ready, I'll request review from you.

[Copilot]

@witchpou I've opened a new pull request, #103, to work on those changes. Once the pull request is ready, I'll request review from you.

[Copilot]

@witchpou I've opened a new pull request, #104, to work on those changes. Once the pull request is ready, I'll request review from you.