Add idle body write detection to Netty HTTP client - #6844

Merged

zoewangg merged 5 commits into

masterfrom

zoewang/WriteIdleTimeoutHandler

Apr 10, 2026

SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 10, 2026 in 15m 41s

Quality Gate passed

Issues
13 New issues
0 Accepted issues

Measures
0 Security Hotspots
95.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 67 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqu_&open=AZ1t2zk8-0BjvfN9zqu_&pullRequest=6844

Check failure on line 116 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvF&open=AZ1t2zk8-0BjvfN9zqvF&pullRequest=6844

Check warning on line 22 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/internal/WriteIdleTimeoutHandlerTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'io.netty.handler.timeout.IdleState'.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zkg-0BjvfN9zqu-&open=AZ1t2zkg-0BjvfN9zqu-&pullRequest=6844

Check warning on line 96 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.InterruptedException', as it cannot be thrown from method's body.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvD&open=AZ1t2zk8-0BjvfN9zqvD&pullRequest=6844

Check failure on line 120 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvG&open=AZ1t2zk8-0BjvfN9zqvG&pullRequest=6844

Check failure on line 158 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvI&open=AZ1t2zk8-0BjvfN9zqvI&pullRequest=6844

Check failure on line 51 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/internal/WriteIdleTimeoutHandlerTest.java

sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zkg-0BjvfN9zqu9&open=AZ1t2zkg-0BjvfN9zqu9&pullRequest=6844

Check failure on line 124 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvH&open=AZ1t2zk8-0BjvfN9zqvH&pullRequest=6844

Check warning on line 61 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvC&open=AZ1t2zk8-0BjvfN9zqvC&pullRequest=6844

Check warning on line 36 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.util.concurrent.ExecutionException'.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvJ&open=AZ1t2zk8-0BjvfN9zqvJ&pullRequest=6844

Check warning on line 96 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.util.concurrent.TimeoutException', as it cannot be thrown from method's body.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvE&open=AZ1t2zk8-0BjvfN9zqvE&pullRequest=6844

Check warning on line 82 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvA&open=AZ1t2zk8-0BjvfN9zqvA&pullRequest=6844

Check warning on line 96 in http-clients/netty-nio-client/src/test/java/software/amazon/awssdk/http/nio/netty/fault/WriteIdleTimeoutTest.java

sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1t2zk8-0BjvfN9zqvB&open=AZ1t2zk8-0BjvfN9zqvB&pullRequest=6844

View more details on SonarQubeCloud

Navigation Menu

Uh oh!

Add idle body write detection to Netty HTTP client - #6844