Fix baggage parsing for invalid percent-encoded members

Author: Vcode2407

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

@@ -140,8 +140,13 @@ private void putBaggage(
     if (entriesAdded >= maxEntries) {
       return;
     }
-    String decodedValue = decodeValue(value);
-    metadataValue = decodeValue(metadataValue);
+    String decodedValue;
+    try {
+      decodedValue = decodeValue(value);
+      metadataValue = decodeValue(metadataValue);
+    } catch (IllegalArgumentException e) {
+      return;
+    }
     BaggageEntryMetadata baggageEntryMetadata =
         metadataValue != null
             ? BaggageEntryMetadata.create(metadataValue)

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

@@ -439,6 +439,21 @@ void extract_member_someInvalid() {
     assertThat(Baggage.fromContext(result)).isEqualTo(expectedBaggage);
   }
 
+  @Test
+  void extract_member_invalidPercentEncoding_preservesValidMembers() {
+    W3CBaggagePropagator propagator = W3CBaggagePropagator.getInstance();
+
+    Context result =
+        propagator.extract(
+            Context.root(),
+            ImmutableMap.of("baggage", "key1=value1,bad=va%lue,key2=value2"),
+            getter);
+
+    Baggage expectedBaggage = Baggage.builder().put("key1", "value1").put("key2", "value2").build();
+
+    assertThat(Baggage.fromContext(result)).isEqualTo(expectedBaggage);
+  }
+
   @Test
   void extract_nullContext() {
     assertThat(W3CBaggagePropagator.getInstance().extract(null, Collections.emptyMap(), getter))