Normalize carrier keys in get such all results of keys() return values in get

Author: jack-berg

api/incubator/src/main/java/io/opentelemetry/api/incubator/propagation/EnvironmentGetter.java

@@ -83,7 +83,14 @@ public String get(@Nullable Map<String, String> carrier, String key) {
       return null;
     }
     String normalizedKey = EnvironmentSetter.normalizeKey(key);
-    return carrier.get(normalizedKey);
+    String[] result = new String[] {null};
+    carrier.forEach(
+        (entryKey, entryValue) -> {
+          if (EnvironmentSetter.normalizeKey(entryKey).equals(normalizedKey)) {
+            result[0] = entryValue;
+          }
+        });
+    return result[0];
   }
 
   @Override

api/incubator/src/test/java/io/opentelemetry/api/incubator/propagation/EnvironmentGetterTest.java

@@ -38,7 +38,7 @@ void get() {
   void get_normalization() {
     Map<String, String> carrier = new HashMap<>();
     carrier.put("OTEL_TRACE_ID", "val1");
-    carrier.put("OTEL_BAGGAGE_KEY", "val2");
+    carrier.put("otel-baggage-key", "val2");
 
     assertThat(EnvironmentGetter.getInstance().get(carrier, "otel.trace.id")).isEqualTo("val1");
     assertThat(EnvironmentGetter.getInstance().get(carrier, "otel-baggage-key")).isEqualTo("val2");
@@ -65,6 +65,19 @@ void keys_valuesAreNormalized() {
     logCapturer.assertContains("keys() called on EnvironmentGetter");
   }
 
+  @Test
+  void getReturnsAllForAllKeys() {
+    // For a carrier containing keys that are both normalized and not normalized, verify all results
+    // from keys() return values for get.
+    Map<String, String> carrier = new HashMap<>();
+    carrier.put("otel.trace.id", "V1");
+    carrier.put("OTEL_BAGGAGE_KEY", "V2");
+
+    for (String key : EnvironmentGetter.getInstance().keys(carrier)) {
+      assertThat(EnvironmentGetter.getInstance().get(carrier, key)).isNotNull();
+    }
+  }
+
   @Test
   void get_valuesAreUnmodified() {
     Map<String, String> carrier = new HashMap<>();