fix(gax): make isLoggingEnabled and setLoggingEnabled package private

Author: westarle

gax-java/gax/src/main/java/com/google/api/gax/logging/LoggingUtils.java

@@ -37,47 +37,28 @@
 public class LoggingUtils {
 
   static final String GOOGLE_SDK_JAVA_LOGGING = "GOOGLE_SDK_JAVA_LOGGING";
-  static final String GOOGLE_SDK_JAVA_LOGGING_V2 = "GOOGLE_SDK_JAVA_LOGGING_V2";
 
   private static boolean loggingEnabled = checkLoggingEnabled(GOOGLE_SDK_JAVA_LOGGING);
-  private static boolean loggingV2Enabled = checkLoggingEnabled(GOOGLE_SDK_JAVA_LOGGING_V2);
 
   /**
-   * Returns whether client-side logging is enabled (V1 or V2).
+   * Returns whether client-side logging is enabled.
    *
    * @return true if logging is enabled, false otherwise.
    */
-  public static boolean isLoggingEnabled() {
-    return loggingEnabled || loggingV2Enabled;
-  }
-
-  /**
-   * Returns whether client-side logging V2 (Actionable Errors) is enabled.
-   *
-   * @return true if V2 logging is enabled, false otherwise.
-   */
-  public static boolean isLoggingV2Enabled() {
-    return loggingV2Enabled;
+  static boolean isLoggingEnabled() {
+    return loggingEnabled;
   }
 
   /**
    * Sets whether client-side logging is enabled. Visible for testing.
    *
    * @param enabled true to enable logging, false to disable.
    */
-  public static void setLoggingEnabled(boolean enabled) {
+  @com.google.common.annotations.VisibleForTesting
+  static void setLoggingEnabled(boolean enabled) {
     loggingEnabled = enabled;
   }
 
-  /**
-   * Sets whether client-side logging V2 is enabled. Visible for testing.
-   *
-   * @param enabled true to enable logging, false to disable.
-   */
-  public static void setLoggingV2Enabled(boolean enabled) {
-    loggingV2Enabled = enabled;
-  }
-
   private static boolean checkLoggingEnabled(String envVar) {
     String enableLogging = System.getenv(envVar);
     return "true".equalsIgnoreCase(enableLogging);

gax-java/gax/src/main/java/com/google/api/gax/logging/Slf4jLoggingHelpers.java

@@ -114,11 +114,10 @@ static void logResponse(
     LoggingUtils.executeWithTryCatch(
         () -> {
           Logger logger = loggerProvider.getLogger();
-          boolean isV2 = LoggingUtils.isLoggingV2Enabled();
-          if (!isV2 && logger.isInfoEnabled()) {
+          if (logger.isInfoEnabled()) {
             logDataBuilder.responseStatus(status);
           }
-          if (!isV2 && logger.isInfoEnabled() && !logger.isDebugEnabled()) {
+          if (logger.isInfoEnabled() && !logger.isDebugEnabled()) {
             Map<String, Object> responseData = logDataBuilder.build().toMapResponse();
             Slf4jUtils.log(logger, Level.INFO, responseData, "Received response");
           }

gax-java/gax/src/test/java/com/google/api/gax/logging/LoggingUtilsTest.java

@@ -105,17 +105,17 @@ void testLogActionableError_loggingDisabled() {
 
   @Test
   void testLogActionableError_success() {
-    LoggingUtils.setLoggingV2Enabled(true);
+    LoggingUtils.setLoggingEnabled(true);
     LoggerProvider loggerProvider = mock(LoggerProvider.class);
     Logger logger = mock(Logger.class);
     when(loggerProvider.getLogger()).thenReturn(logger);
 
     org.slf4j.spi.LoggingEventBuilder eventBuilder = mock(org.slf4j.spi.LoggingEventBuilder.class);
-    when(logger.atInfo()).thenReturn(eventBuilder);
+    when(logger.atDebug()).thenReturn(eventBuilder);
     when(eventBuilder.addKeyValue(anyString(), any())).thenReturn(eventBuilder);
 
     Map<String, Object> context = Collections.singletonMap("key", "value");
-    LoggingUtils.logActionableError(context, loggerProvider, org.slf4j.event.Level.INFO, "message");
+    LoggingUtils.logActionableError(context, loggerProvider, org.slf4j.event.Level.DEBUG, "message");
 
     verify(loggerProvider).getLogger();
   }