fix: add guard against the ArrayIndexOutOfBoundsException in BaggageCodec… (#7239)

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,6 @@
 package io.opentelemetry.api.baggage.propagation;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 import java.nio.charset.StandardCharsets;
 import org.junit.jupiter.api.Test;
@@ -35,8 +34,8 @@ void shouldDecodePercentEncodedValues(String percentEncoded, String expectedDeco
   }
 
   @Test
-  void shouldThrowIfMalformedData() {
-    assertThatThrownBy(() -> BaggageCodec.decode("%1", StandardCharsets.UTF_8))
-        .isInstanceOf(IllegalArgumentException.class);
+  void shouldIgnoreIfMalformedData() {
+    assertThat(BaggageCodec.decode("%", StandardCharsets.UTF_8)).isEqualTo("");
+    assertThat(BaggageCodec.decode("%1", StandardCharsets.UTF_8)).isEqualTo("");
   }
 }