fix: guard against the ArrayIndexOutOfBoundsException in BaggageCodec class

Author: rlacksgus97

api/all/src/main/java/io/opentelemetry/api/baggage/propagation/BaggageCodec.java

@@ -44,13 +44,17 @@ private static byte[] decode(byte[] bytes) {
     for (int i = 0; i < bytes.length; i++) {
       int b = bytes[i];
       if (b == ESCAPE_CHAR) {
-        try {
-          int u = digit16(bytes[++i]);
-          int l = digit16(bytes[++i]);
-          buffer.write((char) ((u << 4) + l));
-        } catch (ArrayIndexOutOfBoundsException e) { // FIXME
-          throw new IllegalArgumentException("Invalid URL encoding: ", e);
+        if (i + 1 >= bytes.length) {
+          return buffer.toByteArray();
         }
+        int u = digit16(bytes[++i]);
+
+        if (i + 1 >= bytes.length) {
+          return buffer.toByteArray();
+        }
+        int l = digit16(bytes[++i]);
+
+        buffer.write((char) ((u << 4) + l));
       } else {
         buffer.write(b);
       }

api/all/src/test/java/io/opentelemetry/api/baggage/propagation/BaggageCodecTest.java

@@ -6,7 +6,7 @@
 package io.opentelemetry.api.baggage.propagation;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.assertj.core.api.Assertions.assertThatNoException;
 
 import java.nio.charset.StandardCharsets;
 import org.junit.jupiter.api.Test;
@@ -36,7 +36,6 @@ void shouldDecodePercentEncodedValues(String percentEncoded, String expectedDeco
 
   @Test
   void shouldThrowIfMalformedData() {
-    assertThatThrownBy(() -> BaggageCodec.decode("%1", StandardCharsets.UTF_8))
-        .isInstanceOf(IllegalArgumentException.class);
+    assertThatNoException().isThrownBy(() -> BaggageCodec.decode("%1", StandardCharsets.UTF_8));
   }
 }