Skip to content
This repository was archived by the owner on Sep 14, 2026. It is now read-only.

security: bound room creation and reclaim empty rooms (DoS) - #34

Merged
krotname merged 1 commit into
mainfrom
security/fix-room-exhaustion
Jul 23, 2026
Merged

krotname merged 1 commit into
mainfrom
security/fix-room-exhaustion

Bound room growth with reclamation, a room cap, and rate limits

b5a35dd
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 23, 2026 in 22s

Annotations

Check warning on line 117 in src/main/java/dev/krotname/networkchat/network/ChatServer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code in order to not assign to this loop counter from within the loop body.

See more on https://sonarcloud.io/project/issues?id=krotname_Network-Chat&issues=AZ-NoJXNkL2BOW0UHmRC&open=AZ-NoJXNkL2BOW0UHmRC&pullRequest=34

Check failure on line 51 in src/main/java/dev/krotname/networkchat/network/ChatServerConfig.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=krotname_Network-Chat&issues=AZ-NoJaCkL2BOW0UHmRD&open=AZ-NoJaCkL2BOW0UHmRD&pullRequest=34

Check warning on line 404 in src/main/java/dev/krotname/networkchat/client/ChatClient.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.out by a logger.

See more on https://sonarcloud.io/project/issues?id=krotname_Network-Chat&issues=AZ-NoJaekL2BOW0UHmRE&open=AZ-NoJaekL2BOW0UHmRE&pullRequest=34