Add WWW-Authenticate as an Allowed Logged Header (Azure#26973)

Author: alzimmermsft

sdk/core/azure-core/CHANGELOG.md

@@ -10,6 +10,9 @@
 
 ### Other Changes
 
+- Added `x-ms-request-id`, `MS-CV`, `WWW-Authenticate` as default logged headers and `api-version` as a default logged
+  query parameter.
+
 ## 1.25.0 (2022-02-04)
 
 ### Features Added

sdk/core/azure-core/src/main/java/com/azure/core/http/policy/HttpLogOptions.java

@@ -8,6 +8,7 @@
 import com.azure.core.util.logging.ClientLogger;
 
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Objects;
@@ -32,10 +33,13 @@ public class HttpLogOptions {
     private static final String INVALID_APPLICATION_ID_LENGTH = "'applicationId' length cannot be greater than "
         + MAX_APPLICATION_ID_LENGTH;
     private static final String INVALID_APPLICATION_ID_SPACE = "'applicationId' cannot contain spaces.";
-    private static final List<String> DEFAULT_HEADERS_WHITELIST = Arrays.asList(
+    private static final List<String> DEFAULT_HEADERS_ALLOWLIST = Arrays.asList(
+        "x-ms-request-id",
         "x-ms-client-request-id",
         "x-ms-return-client-request-id",
         "traceparent",
+        "MS-CV",
+
         "Accept",
         "Cache-Control",
         "Connection",
@@ -54,16 +58,21 @@ public class HttpLogOptions {
         "Retry-After",
         "Server",
         "Transfer-Encoding",
-        "User-Agent"
+        "User-Agent",
+        "WWW-Authenticate"
+    );
+
+    private static final List<String> DEFAULT_QUERY_PARAMS_ALLOWLIST = Collections.singletonList(
+        "api-version"
     );
 
     /**
      * Creates a new instance that does not log any information about HTTP requests or responses.
      */
     public HttpLogOptions() {
         logLevel = HttpLogDetailLevel.ENVIRONMENT_HTTP_LOG_DETAIL_LEVEL;
-        allowedHeaderNames = new HashSet<>(DEFAULT_HEADERS_WHITELIST);
-        allowedQueryParamNames = new HashSet<>();
+        allowedHeaderNames = new HashSet<>(DEFAULT_HEADERS_ALLOWLIST);
+        allowedQueryParamNames = new HashSet<>(DEFAULT_QUERY_PARAMS_ALLOWLIST);
         applicationId = null;
     }