Skip to content

Commit effcca7

Browse files
authored
Remove redundant spaces on error logs (#747)
1 parent 19350d7 commit effcca7

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/client/artifactory/services/DownloadBase.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ protected void handleUnsuccessfulResponse(HttpEntity entity) throws IOException
4545
if (statusCode == HttpStatus.SC_NOT_FOUND) {
4646
throw new FileNotFoundException("Unable to find " + downloadPath);
4747
}
48-
log.error("Failed to download from '" + downloadPath + "'");
48+
log.error("Failed to download from '" + downloadPath + "'");
4949
throwException(entity, getStatusCode());
5050
}
5151
}

build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/client/artifactory/services/SetProperties.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@
1818

1919
public class SetProperties extends VoidJFrogService {
2020
public static final String SET_PROPERTIES_ENDPOINT = "api/storage/";
21-
private ArrayListMultimap<String, String> propertiesMap;
21+
private final ArrayListMultimap<String, String> propertiesMap;
2222
private final boolean encodeProperties;
2323
private final String relativePath;
24-
private String propertiesString;
24+
private final String propertiesString;
2525

2626
private SetProperties(String relativePath, String propertiesString, ArrayListMultimap<String, String> propertiesMap, boolean encodeProperties, Log log) {
2727
super(log);
@@ -52,7 +52,7 @@ public HttpRequestBase createRequest() throws IOException {
5252

5353
@Override
5454
protected void handleUnsuccessfulResponse(HttpEntity entity) throws IOException {
55-
log.error("Failed to set properties to '" + relativePath + "'");
55+
log.error("Failed to set properties to '" + relativePath + "'");
5656
throwException(entity, getStatusCode());
5757
}
5858

build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/client/distribution/services/DeleteReleaseBundle.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -65,12 +65,12 @@ public HttpRequestBase createRequest() throws IOException {
6565
protected void setResponse(InputStream stream) throws IOException {
6666
result = getMapper().readValue(stream, DistributeReleaseBundleResponse.class);
6767
log.debug("Deletion response: " + getStatusCode());
68-
log.debug("Response: " + toJsonString(result));
68+
log.debug("Response: " + toJsonString(result));
6969
}
7070

7171
@Override
7272
public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IOException {
73-
log.info(request.isDryRun() ? "[Dry run] " : "" + "Deleting " + name + " / " + version);
73+
log.info(request.isDryRun() ? "[Dry run] " : "Deleting " + name + " / " + version);
7474
super.execute(client);
7575
if (sync && !request.isDryRun()) {
7676
waitForDeletion(client);

build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/client/distribution/services/DistributeReleaseBundle.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public HttpRequestBase createRequest() throws IOException {
6363

6464
@Override
6565
public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IOException {
66-
log.info(request.isDryRun() ? "[Dry run] " : "" + "Distributing " + name + " / " + version);
66+
log.info(request.isDryRun() ? "[Dry run] " : "Distributing " + name + " / " + version);
6767
super.execute(client);
6868
if (sync && !request.isDryRun()) {
6969
waitForDistribution(client);
@@ -75,7 +75,7 @@ public DistributeReleaseBundleResponse execute(JFrogHttpClient client) throws IO
7575
protected void setResponse(InputStream stream) throws IOException {
7676
result = getMapper().readValue(stream, DistributeReleaseBundleResponse.class);
7777
log.debug("Distribution response: " + getStatusCode());
78-
log.debug("Response: " + toJsonString(result));
78+
log.debug("Response: " + toJsonString(result));
7979
}
8080

8181
private void waitForDistribution(JFrogHttpClient client) throws IOException {

build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/client/distribution/services/VoidDistributionService.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,6 @@ protected VoidDistributionService(Log log) {
1919
@Override
2020
protected void setResponse(InputStream stream) throws IOException {
2121
String ResponseMessage = getStatusCode() + IOUtils.toString(stream, StandardCharsets.UTF_8.name());
22-
log.debug("Distribution response: " + ResponseMessage);
22+
log.debug("Distribution response: " + ResponseMessage);
2323
}
2424
}

0 commit comments

Comments
 (0)