Fix Cr comments

jfrog · Nov 26, 2023 · 7c7a6f5 · 7c7a6f5
1 parent ba75b2e
commit 7c7a6f5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 20 deletions.
diff --git a/src/main/java/com/jfrog/ide/idea/scan/ScanBinaryExecutor.java b/src/main/java/com/jfrog/ide/idea/scan/ScanBinaryExecutor.java
@@ -167,11 +167,7 @@ protected List<JFrogSecurityWarning> execute(ScanConfig.Builder inputFileBuilder
             //  As it is an internal binary execution, the message should be printed for DEBUG use only.
             indicator.setText(String.format("Running %s scan at %s", scanType.toString().toLowerCase(), String.join(" ", inputParams.getRoots())));
             String cmd = String.format("%s %s", binaryTargetPath.toString(), join(" ", args));
-            log.info(String.format("""
-                    Executing JAS scanner
-                    %s
-                    with config:
-                    %s""", cmd, inputParams));
+            log.info(String.format("Executing JAS scanner %s with config: %s", cmd, inputParams));
             CommandExecutor commandExecutor = new CommandExecutor(binaryTargetPath.toString(), createEnvWithCredentials());
             CommandResults commandResults = commandExecutor.exeCommand(binaryTargetPath.toFile().getParentFile(), args,
                     null, new NullLog(), Long.MAX_VALUE, TimeUnit.MINUTES);
@@ -186,7 +182,7 @@ protected List<JFrogSecurityWarning> execute(ScanConfig.Builder inputFileBuilder
             log.info(String.format("Failed to run command: %s", cmd));
             switch (commandResults.getExitValue()) {
                 case USER_NOT_ENTITLED -> {
-                    log.info("User not entitled for advance security scan");
+                    log.debug("User not entitled for advance security scan");
                     return List.of();
                 }
                 case NOT_SUPPORTED -> {

diff --git a/src/main/java/com/jfrog/ide/idea/scan/data/ScanConfig.java b/src/main/java/com/jfrog/ide/idea/scan/data/ScanConfig.java
@@ -3,11 +3,13 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.jfrog.ide.common.nodes.subentities.SourceCodeScanType;
 import lombok.Getter;
+import lombok.ToString;
 
 import java.util.ArrayList;
 import java.util.List;
 
 @Getter
+@ToString
 public class ScanConfig {
     @JsonProperty("type")
     private SourceCodeScanType scanType;
@@ -95,20 +97,6 @@ public void setSkippedFolders(List<String> skippedFolders) {
         this.skippedFolders = skippedFolders;
     }
 
-    @Override
-    public String toString() {
-        return "ScanConfig{" +
-                "scanType=" + scanType +
-                ", language='" + language + '\'' +
-                ", roots=" + roots +
-                ", output='" + output + '\'' +
-                ", grepDisable=" + grepDisable +
-                ", cves=" + cves +
-                ", skippedFolders=" + skippedFolders +
-                ", excludedRules=" + excludedRules +
-                '}';
-    }
-
     public static class Builder {
         private SourceCodeScanType scanType;
         private String language;