Handle gemini review comments.

Author: sauravzg

xds/src/main/java/io/grpc/xds/internal/headermutations/HeaderMutationFilter.java

@@ -97,8 +97,9 @@ private boolean isHeaderMutationAllowed(String headerName,
         && rules.disallowExpression().get().matcher(headerName).matches()) {
       return false;
     }
-    if (rules.allowExpression().isPresent()) {
-      return rules.allowExpression().get().matcher(headerName).matches();
+    if (rules.allowExpression().isPresent()
+        && rules.allowExpression().get().matcher(headerName).matches()) {
+      return true;
     }
     return !rules.disallowAll();
   }

xds/src/test/java/io/grpc/xds/internal/headermutations/HeaderMutationFilterTest.java

@@ -154,11 +154,31 @@ public void filter_allowExpression_onlyAllowsRelevantExpressions()
     HeaderMutations filtered = filter.filter(mutations);
 
 
-    assertThat(filtered.headersToRemove()).containsExactly("x-allowed-remove");
+    assertThat(filtered.headersToRemove()).containsExactly("x-allowed-remove", "not-allowed-remove");
     assertThat(filtered.headers()).containsExactly(header("x-allowed-key", "value"),
-            header("x-allowed-resp-key", "value"));
+        header("not-allowed-key", "value"),
+        header("x-allowed-resp-key", "value"),
+        header("not-allowed-resp-key", "value"));
+  }
+
+  @Test
+  public void filter_allowExpression_fallsThroughToDisallowAll()
+      throws HeaderMutationDisallowedException {
+    HeaderMutationRulesConfig rules = HeaderMutationRulesConfig.builder()
+        .disallowAll(true)
+        .allowExpression(Pattern.compile("^x-allowed-.*")).build();
+    HeaderMutationFilter filter = new HeaderMutationFilter(Optional.of(rules));
+    HeaderMutations mutations = HeaderMutations.create(
+        ImmutableList.of(header("x-allowed-key", "value"), header("not-allowed-key", "value")),
+        ImmutableList.of("x-allowed-remove", "not-allowed-remove"));
+
+    HeaderMutations filtered = filter.filter(mutations);
+
+    assertThat(filtered.headersToRemove()).containsExactly("x-allowed-remove");
+    assertThat(filtered.headers()).containsExactly(header("x-allowed-key", "value"));
   }
 
+
   @Test
   public void filter_allowExpression_overridesDisallowAll()
       throws HeaderMutationDisallowedException {