checkstyle

Author: mwwoda

src/main/java/com/box/sdk/BoxAPIConnection.java

@@ -31,10 +31,10 @@
 import okhttp3.Authenticator;
 import okhttp3.Call;
 import okhttp3.Credentials;
+import okhttp3.Headers;
 import okhttp3.OkHttpClient;
 import okhttp3.Request;
 import okhttp3.Response;
-import okhttp3.Headers;
 
 /**
  * Represents an authenticated connection to the Box API.
@@ -1266,7 +1266,8 @@ private Response executeOnClient(OkHttpClient httpClient, Request request) {
         try {
             return createNewCall(httpClient, request).execute();
         } catch (IOException e) {
-            throw new BoxAPIException("Couldn't connect to the Box API due to a network error. Request\n" + toSanitizedRequest(request), e);
+            throw new BoxAPIException("Couldn't connect to the Box API due to a network error. Request\n"
+                + toSanitizedRequest(request), e);
         }
     }
 

src/main/java/com/box/sdk/BoxSensitiveDataSanitizer.java

@@ -1,46 +1,38 @@
 package com.box.sdk;
 
+import java.util.Arrays;
+import java.util.HashSet;
+import java.util.Set;
 import okhttp3.Headers;
+import org.jetbrains.annotations.NotNull;
 
-import java.util.Set;
-import java.util.HashSet;
-import java.util.Arrays;
+final class BoxSensitiveDataSanitizer {
+    private static final Set<String> SENSITIVE_KEYS = new HashSet<>(Arrays.asList("authorization", "access_token",
+        "refresh_token", "subject_token", "token", "client_id", "client_secret", "code", "shared_link", "download_url",
+        "jwt_private_key", "jwt_private_key_passphrase", "password"));
 
-class BoxSensitiveDataSanitizer {
-	private static final Set<String> sensitiveKeys = new HashSet<>(Arrays.asList(
-		"authorization",
-		"access_token",
-		"refresh_token",
-		"subject_token",
-		"token",
-		"client_id",
-		"client_secret",
-		"code",
-		"shared_link",
-		"download_url",
-		"jwt_private_key",
-		"jwt_private_key_passphrase",
-		"password"
-	));
+    private BoxSensitiveDataSanitizer() {
+    }
 
-	public static Headers sanitizeHeaders(Headers originalHeaders) {
-		Headers.Builder sanitizedHeadersBuilder = originalHeaders.newBuilder();
+    @NotNull
+    public static Headers sanitizeHeaders(Headers originalHeaders) {
+        Headers.Builder sanitizedHeadersBuilder = originalHeaders.newBuilder();
 
-		for (String originalHeaderName : originalHeaders.names()) {
-			if (isSensitiveKey(originalHeaderName)) {
-				sanitizedHeadersBuilder.set(originalHeaderName, "[REDACTED]");
-			} else {
-				String headerValue = originalHeaders.get(originalHeaderName);
-				if(headerValue != null) {
-					sanitizedHeadersBuilder.set(originalHeaderName, headerValue);
-				}
-			}
-		}
+        for (String originalHeaderName : originalHeaders.names()) {
+            if (isSensitiveKey(originalHeaderName)) {
+                sanitizedHeadersBuilder.set(originalHeaderName, "[REDACTED]");
+            } else {
+                String headerValue = originalHeaders.get(originalHeaderName);
+                if (headerValue != null) {
+                    sanitizedHeadersBuilder.set(originalHeaderName, headerValue);
+                }
+            }
+        }
 
-		return sanitizedHeadersBuilder.build();
-	}
+        return sanitizedHeadersBuilder.build();
+    }
 
-	private static boolean isSensitiveKey(String key) {
-		return sensitiveKeys.contains(key.toLowerCase());
-	}
+    private static boolean isSensitiveKey(@NotNull String key) {
+        return SENSITIVE_KEYS.contains(key.toLowerCase());
+    }
 }