Skip to content

Enable default connection health monitoring for CRT HTTP clients - #6818

Merged
zoewangg merged 1 commit into
masterfrom
zoewang/enableHttpMonitoringByDefault
Mar 31, 2026
Merged

Enable default connection health monitoring for CRT HTTP clients#6818
zoewangg merged 1 commit into
masterfrom
zoewang/enableHttpMonitoringByDefault

Enable default connection health monitoring for CRT HTTP clients

4920601
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 26, 2026 in 15m 53s

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 263 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9T&open=AZ0nexJwApNLOrO5Ng9T&pullRequest=6818

Check warning on line 289 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unnecessary boolean literal.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9U&open=AZ0nexJwApNLOrO5Ng9U&pullRequest=6818

Check warning on line 70 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()".

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexNGApNLOrO5Ng9Z&open=AZ0nexNGApNLOrO5Ng9Z&pullRequest=6818

Check warning on line 159 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this instance-reference to a static reference.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9Q&open=AZ0nexJwApNLOrO5Ng9Q&pullRequest=6818

Check warning on line 27 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexNGApNLOrO5Ng9c&open=AZ0nexNGApNLOrO5Ng9c&pullRequest=6818

Check warning on line 222 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove useless curly braces around statement

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9R&open=AZ0nexJwApNLOrO5Ng9R&pullRequest=6818

Check warning on line 28 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ0nexNGApNLOrO5Ng9d&open=AZ0nexNGApNLOrO5Ng9d&pullRequest=6818

Check warning on line 68 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "socket" local variable.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexNGApNLOrO5Ng9Y&open=AZ0nexNGApNLOrO5Ng9Y&pullRequest=6818

Check warning on line 103 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ0nexNGApNLOrO5Ng9a&open=AZ0nexNGApNLOrO5Ng9a&pullRequest=6818

Check warning on line 103 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/NonResponsiveServerTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ0nexNGApNLOrO5Ng9b&open=AZ0nexNGApNLOrO5Ng9b&pullRequest=6818

Check warning on line 245 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call "Optional#isPresent()" before accessing the value.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9P&open=AZ0nexJwApNLOrO5Ng9P&pullRequest=6818

Check warning on line 204 in services/s3/src/test/java/software/amazon/awssdk/services/s3/internal/multipart/KnownContentLengthAsyncRequestBodySubscriberTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "subscriber" which hides the field declared at line 66.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexKXApNLOrO5Ng9X&open=AZ0nexKXApNLOrO5Ng9X&pullRequest=6818

Check warning on line 229 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "contentLength" which hides the field declared at line 119.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9S&open=AZ0nexJwApNLOrO5Ng9S&pullRequest=6818

Check warning on line 126 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "maximumChunkSizeInByte" private field.

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexJwApNLOrO5Ng9V&open=AZ0nexJwApNLOrO5Ng9V&pullRequest=6818

Check warning on line 272 in services/s3/src/test/java/software/amazon/awssdk/services/s3/internal/multipart/KnownContentLengthAsyncRequestBodySubscriberTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "existingParts".

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0nexKXApNLOrO5Ng9W&open=AZ0nexKXApNLOrO5Ng9W&pullRequest=6818