Fix for the following issue:

Logger calls should be surrounded by log level guards.
davejohnston · May 29, 2024 · 21c9b1a · 21c9b1a
1 parent f9ebacf
commit 21c9b1a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/java/io/harness/cf/client/api/InnerClient.java b/src/main/java/io/harness/cf/client/api/InnerClient.java
@@ -72,7 +72,7 @@ public InnerClient(@NonNull final String sdkKey, @NonNull final BaseConfig optio
   }
 
   public InnerClient(@NonNull final Connector connector) {
-    this(connector, BaseConfig.builder().build());
+this(connector, BaseConfig.builder().build());
   }
 
   public InnerClient(@NonNull Connector connector, @NonNull final BaseConfig options) {
@@ -81,7 +81,9 @@ public InnerClient(@NonNull Connector connector, @NonNull final BaseConfig optio
 
   protected void setUp(@NonNull final Connector connector, @NonNull final BaseConfig options) {
     this.options = options;
-    log.info("Starting SDK client with configuration: {}", this.options);
+    if (log.isInfoEnabled()) {
+      log.info("Starting SDK client with configuration: {}", this.options);
+    }
     this.connector = connector;
     this.connector.setOnUnauthorized(this::onUnauthorized);