Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
85ec706
Update dependency com.puppycrawl.tools:checkstyle to v11.0.1
xdev-renovate Sep 1, 2025
a9590eb
Merge pull request #194 from xdev-software/renovate/com.puppycrawl.to…
AB-xdev Sep 1, 2025
ef49d71
Merge branch 'master' into update-from-template-merged
xdev-gh-bot Sep 1, 2025
91e0eb1
Update dependency org.openapitools:jackson-databind-nullable to v0.2.7
xdev-renovate Sep 2, 2025
e7e9229
Update dependency com.fasterxml.jackson:jackson-bom to v2.20.0
xdev-renovate Sep 2, 2025
c54c909
Update dependency org.openapitools:openapi-generator-maven-plugin to …
xdev-renovate Sep 2, 2025
ecd39c7
Merge pull request #292 from xdev-software/renovate/org.openapitools-…
AB-xdev Sep 2, 2025
9484b57
Merge pull request #303 from xdev-software/renovate/com.fasterxml.jac…
AB-xdev Sep 2, 2025
d3be0f5
Merge pull request #295 from xdev-software/renovate/org.openapitools-…
AB-xdev Sep 2, 2025
c41665b
PMD: AvoidUnmanagedThreads
AB-xdev Sep 2, 2025
3b31f0b
Fix format
AB-xdev Sep 2, 2025
ebe579f
PMD: Add PostConstruct and PreDestroy
AB-xdev Sep 2, 2025
3792ace
Merge branch 'master' into update-from-template-merged
xdev-gh-bot Sep 2, 2025
a2ffc8c
Merge branch 'master' into update-from-template-merged
xdev-gh-bot Sep 2, 2025
d23faec
Init
AB-xdev Sep 3, 2025
b967a6a
Merge remote-tracking branch 'origin/update-from-template-xdev-softwa…
AB-xdev Sep 3, 2025
a42d3cb
Merge branch 'master' into update-from-template-xdev-software/java-te…
xdev-gh-bot Sep 3, 2025
26af372
Merge branch 'master' into update-from-template-xdev-software/standar…
xdev-gh-bot Sep 3, 2025
bba9aa6
No EoL
AB-xdev Sep 3, 2025
dc3ac3d
Merge branch 'master' into update-from-template-xdev-software/java-se…
xdev-gh-bot Sep 3, 2025
1931201
Merge branch 'master' into update-from-template-xdev-software/java-te…
xdev-gh-bot Sep 3, 2025
2296901
Merge branch 'master' into update-from-template-xdev-software/standar…
xdev-gh-bot Sep 3, 2025
ab76db3
Merge branch 'develop' into update-from-template-merged
xdev-gh-bot Sep 8, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 65 additions & 0 deletions .config/pmd/java/ruleset.xml
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,71 @@
</properties>
</rule>

<rule name="AvoidPostConstruct"
language="java"
message="Avoid @PostConstruct"
class="net.sourceforge.pmd.lang.rule.xpath.XPathRule">
<description>
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.
</description>
<priority>3</priority>
<properties>
<property name="xpath">
<value>
<![CDATA[
//MethodDeclaration[pmd-java:hasAnnotation('jakarta.annotation.PostConstruct')]
]]>
</value>
</property>
</properties>
</rule>

<rule name="AvoidPreDestroy"
language="java"
message="Avoid @PreDestroy"
class="net.sourceforge.pmd.lang.rule.xpath.XPathRule">
<description>
`@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.
</description>
<priority>3</priority>
<properties>
<property name="xpath">
<value>
<![CDATA[
//MethodDeclaration[pmd-java:hasAnnotation('jakarta.annotation.PreDestroy')]
]]>
</value>
</property>
</properties>
</rule>

<rule name="AvoidUnmanagedThreads"
language="java"
message="Avoid unmanaged threads"
class="net.sourceforge.pmd.lang.rule.xpath.XPathRule">
<description>
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.
</description>
<priority>3</priority>
<properties>
<property name="xpath">
<value>
<![CDATA[
//MethodCall[pmd-java:matchesSig('java.lang.Thread#start()') or pmd-java:matchesSig('java.lang.Thread#startVirtualThread(java.lang.Runnable)') or pmd-java:matchesSig('java.lang.Thread$Builder#start(java.lang.Runnable)')]
]]>
</value>
</property>
</properties>
</rule>

<rule name="JavaObjectSerializationIsUnsafe"
language="java"
message="Using Java Object (De-)Serialization is unsafe and has led to too many security vulnerabilities"
Expand Down
2 changes: 1 addition & 1 deletion .idea/codeStyles/Project.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
<dependency>
<groupId>com.puppycrawl.tools</groupId>
<artifactId>checkstyle</artifactId>
<version>11.0.0</version>
<version>11.0.1</version>
</dependency>
</dependencies>
<configuration>
Expand Down
8 changes: 4 additions & 4 deletions sessionize-java-client/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@
<dependency>
<groupId>com.fasterxml.jackson</groupId>
<artifactId>jackson-bom</artifactId>
<version>2.19.2</version>
<version>2.20.0</version>
<type>pom</type>
<scope>import</scope>
</dependency>
Expand Down Expand Up @@ -96,7 +96,7 @@
<dependency>
<groupId>org.openapitools</groupId>
<artifactId>jackson-databind-nullable</artifactId>
<version>0.2.6</version>
<version>0.2.7</version>
</dependency>

<dependency>
Expand Down Expand Up @@ -324,7 +324,7 @@
<plugin>
<groupId>org.openapitools</groupId>
<artifactId>openapi-generator-maven-plugin</artifactId>
<version>7.14.0</version>
<version>7.15.0</version>
<executions>
<execution>
<goals>
Expand Down Expand Up @@ -449,7 +449,7 @@
<dependency>
<groupId>com.puppycrawl.tools</groupId>
<artifactId>checkstyle</artifactId>
<version>11.0.0</version>
<version>11.0.1</version>
</dependency>
</dependencies>
<configuration>
Expand Down