Skip to content

Split CBOR encoder to support Jackson 2 and 3 #134

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
alex-w0 wants to merge 7 commits into
base: axon-4
Choose a base branch
from

swap jackson encoding strategy order

3655005
Select commit
Loading
Failed to load commit list.
Open

Split CBOR encoder to support Jackson 2 and 3 #134

swap jackson encoding strategy order
3655005
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 1, 2026 in 48s

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 50 in console-framework-client/src/main/java/io/axoniq/console/framework/api/AxoniqConsoleMeasuringHandlerInterceptor.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless non-null assertion !!, it always succeeds. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k7W0Hy_-YdDj5U3&open=AZ1I8k7W0Hy_-YdDj5U3&pullRequest=134

Check warning on line 171 in console-framework-client/src/main/java/io/axoniq/console/framework/eventprocessor/DeadLetterManager.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace function call with indexed accessor. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5s0Hy_-YdDj5Uf&open=AZ1I8k5s0Hy_-YdDj5Uf&pullRequest=134

Check warning on line 179 in console-framework-client-api/src/main/java/io/axoniq/console/framework/api/clientIdentification.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k4z0Hy_-YdDj5UY&open=AZ1I8k4z0Hy_-YdDj5UY&pullRequest=134

Check warning on line 44 in console-framework-client/src/main/java/io/axoniq/console/framework/client/strategy/CborJackson2EncodingStrategy.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unchecked cast. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6S0Hy_-YdDj5Uo&open=AZ1I8k6S0Hy_-YdDj5Uo&pullRequest=134

Check warning on line 210 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Uy&open=AZ1I8k6s0Hy_-YdDj5Uy&pullRequest=134

Check warning on line 339 in console-framework-client/src/main/java/io/axoniq/console/framework/AxoniqConsoleConfigurerModule.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1JhkgpXykSk5tqA81N&open=AZ1JhkgpXykSk5tqA81N&pullRequest=134

Check warning on line 56 in console-framework-client/src/main/java/io/axoniq/console/framework/eventprocessor/EventProcessorManager.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace function call with indexed accessor. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k520Hy_-YdDj5Uj&open=AZ1I8k520Hy_-YdDj5Uj&pullRequest=134

Check warning on line 184 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Ut&open=AZ1I8k6s0Hy_-YdDj5Ut&pullRequest=134

Check warning on line 63 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleWrappedEventScheduler.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace with interface delegation using "by" in the class header. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6j0Hy_-YdDj5Uq&open=AZ1I8k6j0Hy_-YdDj5Uq&pullRequest=134

Check warning on line 207 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Ux&open=AZ1I8k6s0Hy_-YdDj5Ux&pullRequest=134

Check warning on line 48 in console-framework-client/src/main/java/io/axoniq/console/framework/application/DomainEventStreamProvider.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unchecked cast. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5X0Hy_-YdDj5Ud&open=AZ1I8k5X0Hy_-YdDj5Ud&pullRequest=134

Check warning on line 165 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Ur&open=AZ1I8k6s0Hy_-YdDj5Ur&pullRequest=134

Check warning on line 37 in console-framework-client/src/main/java/io/axoniq/console/framework/application/DomainEventStreamProvider.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unchecked cast. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5X0Hy_-YdDj5Uc&open=AZ1I8k5X0Hy_-YdDj5Uc&pullRequest=134

Check warning on line 203 in console-framework-client/src/main/java/io/axoniq/console/framework/eventprocessor/DeadLetterManager.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace function call with indexed accessor. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5s0Hy_-YdDj5Uh&open=AZ1I8k5s0Hy_-YdDj5Uh&pullRequest=134

Check warning on line 50 in console-framework-client/src/main/java/io/axoniq/console/framework/api/AxoniqConsoleMeasuringHandlerInterceptor.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless non-null assertion !!, it always succeeds. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k7W0Hy_-YdDj5U4&open=AZ1I8k7W0Hy_-YdDj5U4&pullRequest=134

Check warning on line 108 in console-framework-client/src/main/java/io/axoniq/console/framework/eventprocessor/EventProcessorManager.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k520Hy_-YdDj5Ui&open=AZ1I8k520Hy_-YdDj5Ui&pullRequest=134

Check warning on line 175 in console-framework-client-api/src/main/java/io/axoniq/console/framework/api/clientIdentification.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k4z0Hy_-YdDj5UW&open=AZ1I8k4z0Hy_-YdDj5UW&pullRequest=134

Check warning on line 419 in console-framework-client/src/main/java/io/axoniq/console/framework/client/SetupPayloadCreator.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace function call with indexed accessor. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6C0Hy_-YdDj5Um&open=AZ1I8k6C0Hy_-YdDj5Um&pullRequest=134

Check warning on line 199 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Uw&open=AZ1I8k6s0Hy_-YdDj5Uw&pullRequest=134

Check warning on line 191 in console-framework-client/src/main/java/io/axoniq/console/framework/eventprocessor/DeadLetterManager.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace function call with indexed accessor. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5s0Hy_-YdDj5Ug&open=AZ1I8k5s0Hy_-YdDj5Ug&pullRequest=134

Check warning on line 45 in console-framework-client/src/main/java/io/axoniq/console/framework/client/strategy/CborJackson3EncodingStrategy.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unchecked cast. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6a0Hy_-YdDj5Up&open=AZ1I8k6a0Hy_-YdDj5Up&pullRequest=134

Check warning on line 89 in console-framework-client/src/main/java/io/axoniq/console/framework/utils.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this if expression with require(maxBytes >= messageBytes.size) { "maxBytes ($maxBytes) must be at least ${messageBytes.size} bytes to accommodate truncation message" }. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k7N0Hy_-YdDj5U2&open=AZ1I8k7N0Hy_-YdDj5U2&pullRequest=134

Check warning on line 47 in console-framework-client/src/main/java/io/axoniq/console/framework/application/ApplicationMetricReporter.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Deprecated code should not be used. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k5g0Hy_-YdDj5Ue&open=AZ1I8k5g0Hy_-YdDj5Ue&pullRequest=134

Check warning on line 181 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleSpanFactory.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Map values should be accessed safely. Using the non-null assertion operator here can throw a NullPointerException. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k6s0Hy_-YdDj5Us&open=AZ1I8k6s0Hy_-YdDj5Us&pullRequest=134

Check warning on line 37 in console-framework-client/src/main/java/io/axoniq/console/framework/messaging/AxoniqConsoleWrappedEventStore.kt

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace with interface delegation using "by" in the class header. 

See more on https://sonarcloud.io/project/issues?id=AxonIQ_console-framework-client&issues=AZ1I8k600Hy_-YdDj5Uz&open=AZ1I8k600Hy_-YdDj5Uz&pullRequest=134
View more details on SonarQubeCloud