diff --git a/.config/pmd/java/ruleset.xml b/.config/pmd/java/ruleset.xml index 02a037f..ebdbd83 100644 --- a/.config/pmd/java/ruleset.xml +++ b/.config/pmd/java/ruleset.xml @@ -218,6 +218,71 @@ + + + Using a `@PostConstruct` method is usually only done when field injection is used and initialization needs to be performed after that. + + It's better to do this directly in the constructor with constructor injection, so that all logic will be encapsulated there. + This also makes using the bean in environments where JavaEE is not present - for example in tests - a lot easier, as forgetting to call the `@PostConstruct` method is no longer possible. + + 3 + + + + + + + + + + + + `@PreDestroy` should be replaced by implementing `AutoCloseable` and overwriting the `close` method instead. + + This also makes using the bean in environments where JavaEE is not present - for example in tests - a lot easier, as forgetting to call the `@PreDestroy` method is no much more difficult. + + 3 + + + + + + + + + + + + Trying to manually manage threads usually gets quickly out of control and may result in various problems like uncontrollable spawning of threads. + Threads can also not be cancelled properly. + + Use managed Thread services like `ExecutorService` and `CompletableFuture` instead. + + 3 + + + + + + + + + - + \ No newline at end of file diff --git a/pom.xml b/pom.xml index 1747f10..a2c8121 100644 --- a/pom.xml +++ b/pom.xml @@ -45,7 +45,7 @@ com.puppycrawl.tools checkstyle - 11.0.0 + 11.0.1 diff --git a/sessionize-java-client/pom.xml b/sessionize-java-client/pom.xml index ac123dc..c803ee8 100644 --- a/sessionize-java-client/pom.xml +++ b/sessionize-java-client/pom.xml @@ -61,7 +61,7 @@ com.fasterxml.jackson jackson-bom - 2.19.2 + 2.20.0 pom import @@ -96,7 +96,7 @@ org.openapitools jackson-databind-nullable - 0.2.6 + 0.2.7 @@ -324,7 +324,7 @@ org.openapitools openapi-generator-maven-plugin - 7.14.0 + 7.15.0 @@ -449,7 +449,7 @@ com.puppycrawl.tools checkstyle - 11.0.0 + 11.0.1