Bump org.asynchttpclient:async-http-client from 2.12.3 to 3.0.0 (#384)

Bumps [org.asynchttpclient:async-http-client](https://github.com/AsyncHttpClient/async-http-client) from 2.12.3 to 3.0.0.
- [Release notes](https://github.com/AsyncHttpClient/async-http-client/releases)
- [Changelog](https://github.com/AsyncHttpClient/async-http-client/blob/main/CHANGES.md)
- [Commits](AsyncHttpClient/async-http-client@async-http-client-project-2.12.3...async-http-client-project-3.0.0)

---
updated-dependencies:
- dependency-name: org.asynchttpclient:async-http-client
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Author: dependabot[bot]

pom.xml

@@ -108,7 +108,7 @@
     <apache.httpcore.version>4.4.16</apache.httpcore.version>
     <arpnetworking.commons.version>3.1.1</arpnetworking.commons.version>
     <aspectjrt.version>1.9.22.1</aspectjrt.version>
-    <asynchttpclient.version>2.12.3</asynchttpclient.version>
+    <asynchttpclient.version>3.0.0</asynchttpclient.version>
     <awaitility.version>4.2.1</awaitility.version>
     <commons.codec.version>1.17.1</commons.codec.version>
     <ebean.version>15.5.0</ebean.version>

src/main/java/com/arpnetworking/tsdcore/sinks/HttpSinkActor.java

@@ -199,13 +199,15 @@ public Receive createReceive() {
                     final int attempt = rejected.getAttempt();
                     final Response response = rejected.getResponse();
                     if (_retryableStatusCodes.contains(response.getStatusCode()) && attempt < _sink.getMaximumAttempts()) {
+                        final byte[] requestBodyBytes = rejected.getRequestEntry().getRequest().getByteData();
+                        // CHECKSTYLE.OFF: IllegalInstantiation - This is ok for String from byte[]
+                        final String requestBody = requestBodyBytes == null ? null : new String(requestBodyBytes, Charsets.UTF_8);
+                        // CHECKSTYLE.ON: IllegalInstantiation
                         POST_RETRY_LOGGER.warn()
                             .setMessage("Attempt rejected")
                             .addData("sink", _sink)
                             .addData("status", response.getStatusCode())
-                            // CHECKSTYLE.OFF: IllegalInstantiation - This is ok for String from byte[]
-                            .addData("request", new String(rejected.getRequestEntry().getRequest().getByteData(), Charsets.UTF_8))
-                            // CHECKSTYLE.ON: IllegalInstantiation
+                            .addData("request", requestBody)
                             .addData("response", response.getResponseBody())
                             .addContext("actor", self())
                             .log();
@@ -294,12 +296,16 @@ private void processRejectedRequest(final PostRejected rejected) {
                     responseStatusClass == i ? 1 : 0);
         }
 
+        final byte[] requestBodyBytes = rejected.getRequestEntry().getRequest().getByteData();
+        // CHECKSTYLE.OFF: IllegalInstantiation - This is ok for String from byte[]
+        final String requestBody = requestBodyBytes == null ? null : new String(requestBodyBytes, Charsets.UTF_8);
+        // CHECKSTYLE.ON: IllegalInstantiation
         POST_ERROR_LOGGER.warn()
                 .setMessage("Post rejected")
                 .addData("sink", _sink)
                 .addData("status", responseStatusCode)
                 // CHECKSTYLE.OFF: IllegalInstantiation - This is ok for String from byte[]
-                .addData("request", new String(rejected.getRequestEntry().getRequest().getByteData(), Charsets.UTF_8))
+                .addData("request", requestBody)
                 // CHECKSTYLE.ON: IllegalInstantiation
                 .addData("response", responseBody)
                 .addContext("actor", self())