[UNDERTOW-2405] The correct error code to be sent if frame parser ref…

…uses more data is PROTOCOL_ERROR

This makes this code consistent with the handling of headers that surpass the max header size limit elsewhere in HTTP2 (see Http2HeaderBlockParser.emitHeader), and the justification is that the max header size must have been handshaken with the peer as part of settings frame, via the SETTINGS_HEADER_TABLE_SIZE parameter.

Signed-off-by: Flavia Rainone <frainone@redhat.com>

core/src/main/java/io/undertow/protocols/http2/Http2FrameHeaderParser.java

@@ -26,7 +26,6 @@
 import java.nio.ByteBuffer;
 
 import static io.undertow.protocols.http2.Http2Channel.DATA_FLAG_END_STREAM;
-import static io.undertow.protocols.http2.Http2Channel.ERROR_ENHANCE_YOUR_CALM;
 import static io.undertow.protocols.http2.Http2Channel.FRAME_TYPE_CONTINUATION;
 import static io.undertow.protocols.http2.Http2Channel.FRAME_TYPE_DATA;
 import static io.undertow.protocols.http2.Http2Channel.FRAME_TYPE_GOAWAY;
@@ -110,7 +109,7 @@ public boolean handle(final ByteBuffer byteBuffer) throws IOException {
                     }
                     parser = continuationParser;
                     if (!continuationParser.moreData(length)) {
-                        http2Channel.sendGoAway(ERROR_ENHANCE_YOUR_CALM);
+                        http2Channel.sendGoAway(Http2Channel.ERROR_PROTOCOL_ERROR);
                     }
                     break;
                 }