Remove ignored and excluded headers

Author: pc9795

src/main/java/com/exceptionless/exceptionlessclient/plugins/preconfigured/RequestInfoPlugin.java

@@ -14,13 +14,24 @@
 import java.net.URI;
 import java.net.UnknownHostException;
 import java.net.http.HttpRequest;
+import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.stream.Collectors;
 
 @Slf4j
 public class RequestInfoPlugin implements EventPluginIF {
   private static final Integer DEFAULT_PRIORITY = 70;
+  private static final Set<String> IGNORED_HEADERS =
+      Set.of(
+          "Authorization",
+          "Cookie",
+          "Host",
+          "Method",
+          "Path",
+          "Proxy-Authorization",
+          "Referer",
+          "User-Agent");
 
   @Builder
   public RequestInfoPlugin() {}
@@ -71,7 +82,7 @@ private RequestInfo getRequestInfo(HttpRequest request, RequestInfoGetArgs args)
             .host(request.uri().getHost())
             .path(request.uri().getPath())
             .port(request.uri().getPort())
-            .headers(request.headers().map());
+            .headers(getHeaders(request, args));
 
     if (args.isIncludeIpAddress()) {
       try {
@@ -95,6 +106,14 @@ private RequestInfo getRequestInfo(HttpRequest request, RequestInfoGetArgs args)
     return builder.build();
   }
 
+  private Map<String, List<String>> getHeaders(HttpRequest request, RequestInfoGetArgs args) {
+    Map<String, List<String>> headers =
+        filterExclusions(request.headers().map(), args.getExclusions());
+    return headers.entrySet().stream()
+        .filter(entry -> !IGNORED_HEADERS.contains(entry.getKey()))
+        .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
+  }
+
   private <X> Map<String, X> filterExclusions(Map<String, X> map, Set<String> exclusions) {
     return map.entrySet().stream()
         .filter(

src/test/java/com/exceptionless/exceptionlessclient/plugins/preconfigured/RequestInfoPluginTest.java

@@ -193,7 +193,15 @@ public void itCanExcludeData() {
             .uri(
                 URI.create(
                     "https://localhost:5000/test-path?query-param-key=query-param-value&exclude-query-param=exclude-value"))
-            .header("Cookie", "cookie1=value1;cookie2=value2;exclude-cookie=exclude-value")
+            .headers(
+                "Cookie",
+                "cookie1=value1;cookie2=value2;exclude-cookie=exclude-value",
+                "test-header",
+                "test-value",
+                "Authorization",
+                "test-auth",
+                "exclude-header",
+                "exclude-value")
             .GET()
             .build();
 
@@ -202,7 +210,7 @@ public void itCanExcludeData() {
             .event(Event.builder().build())
             .context(PluginContext.builder().request(httpRequest).build())
             .build();
-    configuration.addDataExclusions("exclude-query-param", "exclude-cookie");
+    configuration.addDataExclusions("exclude-query-param", "exclude-cookie", "exclude-header");
     PrivateInformationInclusions inclusions = configuration.getPrivateInformationInclusions();
     inclusions.setIpAddress(true);
     inclusions.setCookies(true);
@@ -217,5 +225,6 @@ public void itCanExcludeData() {
         .isEqualTo(Map.of("cookie1", "value1", "cookie2", "value2"));
     assertThat(requestInfo.getQueryString())
         .isEqualTo(Map.of("query-param-key", List.of("query-param-value")));
+    assertThat(requestInfo.getHeaders()).isEqualTo(Map.of("test-header", List.of("test-value")));
   }
 }