removed the blanket RuntimeException catch and now only ignore IllegalArgumentException

Author: arturobernalg

httpcore5-h2/src/main/java/org/apache/hc/core5/http2/impl/nio/AbstractH2StreamMultiplexer.java

@@ -1388,7 +1388,7 @@ private void recordPriorityFromHeaders(final H2Stream stream, final List<? exten
         }
         for (final Header h : headers) {
             if (HttpHeaders.PRIORITY.equalsIgnoreCase(h.getName())) {
-                final PriorityValue pv = parsePriorityValue(h.getValue());
+                final PriorityValue pv = parsePriorityValue(h);
                 if (pv != null) {
                     stream.setPriorityValue(pv);
                 }
@@ -1397,10 +1397,24 @@ private void recordPriorityFromHeaders(final H2Stream stream, final List<? exten
         }
     }
 
+    private PriorityValue parsePriorityValue(final Header header) {
+        if (header == null) {
+            return null;
+        }
+        try {
+            return PriorityParamsParser.parse(header).toValueWithDefaults();
+        } catch (final IllegalArgumentException ignore) {
+            return null;
+        }
+    }
+
     private PriorityValue parsePriorityValue(final String field) {
+        if (field == null) {
+            return null;
+        }
         try {
             return PriorityParamsParser.parse(field).toValueWithDefaults();
-        } catch (final RuntimeException ignore) {
+        } catch (final IllegalArgumentException ignore) {
             return null;
         }
     }
@@ -1452,7 +1466,7 @@ public void submit(final List<Header> headers, final boolean endStream) throws I
                     return;
                 }
                 ensureNotClosed();
-                if (peerNoRfc7540Priorities && streams.isSameSide(id)) {
+                if (peerNoRfc7540Priorities) {
                     for (final Header h : headers) {
                         if (HttpHeaders.PRIORITY.equalsIgnoreCase(h.getName())) {
                             final byte[] ascii = h.getValue() != null
@@ -1466,6 +1480,7 @@ public void submit(final List<Header> headers, final boolean endStream) throws I
                             b.append((byte) (sid >> 8));
                             b.append((byte) sid);
                             b.append(ascii, 0, ascii.length);
+
                             final ByteBuffer pl = ByteBuffer.wrap(b.array(), 0, b.length());
                             final RawFrame priUpd = new RawFrame(FrameType.PRIORITY_UPDATE.getValue(), 0, 0, pl);
                             commitFrameInternal(priUpd);