Remove redundant spaces on error logs (#747)

jfrog · Jul 9, 2023 · effcca7 · effcca7
1 parent 19350d7
commit effcca7
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
diff --git a/...g/jfrog/build/extractor/clientConfiguration/client/artifactory/services/DownloadBase.java b/...g/jfrog/build/extractor/clientConfiguration/client/artifactory/services/DownloadBase.java
@@ -45,7 +45,7 @@ protected void handleUnsuccessfulResponse(HttpEntity entity) throws IOException
         if (statusCode == HttpStatus.SC_NOT_FOUND) {
             throw new FileNotFoundException("Unable to find " + downloadPath);
         }
-        log.error("Failed to download from  '" + downloadPath + "'");
+        log.error("Failed to download from '" + downloadPath + "'");
         throwException(entity, getStatusCode());
     }
 }
diff --git a/.../jfrog/build/extractor/clientConfiguration/client/artifactory/services/SetProperties.java b/.../jfrog/build/extractor/clientConfiguration/client/artifactory/services/SetProperties.java
@@ -18,10 +18,10 @@
 
 public class SetProperties extends VoidJFrogService {
     public static final String SET_PROPERTIES_ENDPOINT = "api/storage/";
-    private ArrayListMultimap<String, String> propertiesMap;
+    private final ArrayListMultimap<String, String> propertiesMap;
     private final boolean encodeProperties;
     private final String relativePath;
-    private String propertiesString;
+    private final String propertiesString;
 
     private SetProperties(String relativePath, String propertiesString, ArrayListMultimap<String, String> propertiesMap, boolean encodeProperties, Log log) {
         super(log);
@@ -52,7 +52,7 @@ public HttpRequestBase createRequest() throws IOException {
 
     @Override
     protected void handleUnsuccessfulResponse(HttpEntity entity) throws IOException {
-        log.error("Failed to set properties to  '" + relativePath + "'");
+        log.error("Failed to set properties to '" + relativePath + "'");
         throwException(entity, getStatusCode());
     }
 

diff --git a/...build/extractor/clientConfiguration/client/distribution/services/DeleteReleaseBundle.java b/...build/extractor/clientConfiguration/client/distribution/services/DeleteReleaseBundle.java
@@ -65,12 +65,12 @@ public HttpRequestBase createRequest() throws IOException {
     protected void setResponse(InputStream stream) throws IOException {
         result = getMapper().readValue(stream, DistributeReleaseBundleResponse.class);
         log.debug("Deletion response: " + getStatusCode());
-        log.debug("Response:  " + toJsonString(result));
+        log.debug("Response: " + toJsonString(result));
     }
 
     @Override
     public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IOException {
-        log.info(request.isDryRun() ? "[Dry run] " : "" + "Deleting " + name + " / " + version);
+        log.info(request.isDryRun() ? "[Dry run] " : "Deleting " + name + " / " + version);
         super.execute(client);
         if (sync && !request.isDryRun()) {
             waitForDeletion(client);

diff --git a/...d/extractor/clientConfiguration/client/distribution/services/DistributeReleaseBundle.java b/...d/extractor/clientConfiguration/client/distribution/services/DistributeReleaseBundle.java
@@ -63,7 +63,7 @@ public HttpRequestBase createRequest() throws IOException {
 
     @Override
     public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IOException {
-        log.info(request.isDryRun() ? "[Dry run] " : "" + "Distributing " + name + " / " + version);
+        log.info(request.isDryRun() ? "[Dry run] " : "Distributing " + name + " / " + version);
         super.execute(client);
         if (sync && !request.isDryRun()) {
             waitForDistribution(client);
@@ -75,7 +75,7 @@ public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IO
     protected void setResponse(InputStream stream) throws IOException {
         result = getMapper().readValue(stream, DistributeReleaseBundleResponse.class);
         log.debug("Distribution response: " + getStatusCode());
-        log.debug("Response:  " + toJsonString(result));
+        log.debug("Response: " + toJsonString(result));
     }
 
     private void waitForDistribution(JFrogHttpClient client) throws IOException {

diff --git a/...d/extractor/clientConfiguration/client/distribution/services/VoidDistributionService.java b/...d/extractor/clientConfiguration/client/distribution/services/VoidDistributionService.java
@@ -19,6 +19,6 @@ protected VoidDistributionService(Log log) {
     @Override
     protected void setResponse(InputStream stream) throws IOException {
         String ResponseMessage = getStatusCode() + IOUtils.toString(stream, StandardCharsets.UTF_8.name());
-        log.debug("Distribution response:  " + ResponseMessage);
+        log.debug("Distribution response: " + ResponseMessage);
     }
 }