Fix revapi gate and guard a test header parse

Scope-ignore the java.annotation.removed diff for Http2Handler.userEventTriggered: Netty's internal @ChannelHandlerMask.Skip no-op marker is correctly dropped once the handler overrides the method to process RST_STREAM. Http2Handler is public final and internal-package, so there is no consumer-facing API change, and the ignore is scoped to this method so any real future change is still flagged. Also wrap Long.parseLong in Http2StreamingBodyFlowControlTest.assertEchoed so a missing/non-numeric header fails cleanly instead of throwing NumberFormatException.

Author: hyperxpro

client/src/test/java/org/asynchttpclient/Http2StreamingBodyFlowControlTest.java

@@ -300,12 +300,21 @@ private AsyncHttpClient http2Client() {
 
     private void assertEchoed(Response response, byte[] expected) {
         assertEquals(200, response.getStatusCode(), "request should have completed");
-        assertEquals(expected.length, Long.parseLong(response.getHeader("x-received-length")),
+        assertEquals(expected.length, parseLongHeader(response, "x-received-length"),
                 "server must receive every uploaded byte");
-        assertEquals(crc32(expected), Long.parseLong(response.getHeader("x-received-crc")),
+        assertEquals(crc32(expected), parseLongHeader(response, "x-received-crc"),
                 "uploaded bytes must arrive intact and in order");
     }
 
+    private static long parseLongHeader(Response response, String name) {
+        String value = response.getHeader(name);
+        try {
+            return Long.parseLong(value);
+        } catch (NumberFormatException e) {
+            return fail("response header '" + name + "' was not a parseable long: " + value, e);
+        }
+    }
+
     // =========================================================================
     // NettyInputStreamBody — large streaming upload via InputStreamBodyGenerator
     // =========================================================================

pom.xml

@@ -465,6 +465,13 @@
                                             {
                                                 "code": "java.class.nonPublicPartOfAPI",
                                                 "justification": "Internal classes exposed via public methods are intentional"
+                                            },
+                                            {
+                                                "code": "java.annotation.removed",
+                                                "old": "method void io.netty.channel.ChannelInboundHandlerAdapter::userEventTriggered(io.netty.channel.ChannelHandlerContext, java.lang.Object) throws java.lang.Exception @ org.asynchttpclient.netty.handler.Http2Handler",
+                                                "new": "method void org.asynchttpclient.netty.handler.Http2Handler::userEventTriggered(io.netty.channel.ChannelHandlerContext, java.lang.Object) throws java.lang.Exception",
+                                                "annotationType": "io.netty.channel.ChannelHandlerMask.Skip",
+                                                "justification": "Http2Handler overrides userEventTriggered to handle RST_STREAM (Http2ResetFrame); Netty's internal @ChannelHandlerMask.Skip no-op marker is correctly dropped because the method now does real work. Http2Handler is public final and internal-package, so there is no binary, source, or behavioral change for consumers. Scoped to this exact method so any real future change to it is still flagged."
                                             }
                                         ]
                                     }