You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by sh...@apache.org on 2022/06/19 10:24:41 UTC
[servicecomb-java-chassis] branch master updated: [SCB-2583] enable checkstyle trailing whitespace (#3097)
This is an automated email from the ASF dual-hosted git repository.
shoothzj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git
The following commit(s) were added to refs/heads/master by this push:
new a7a9411ab [SCB-2583] enable checkstyle trailing whitespace (#3097)
a7a9411ab is described below
commit a7a9411ab9c06d25d69773ed3a608d118cfac628
Author: TingTing Wang <19...@qq.com>
AuthorDate: Sun Jun 19 18:24:36 2022 +0800
[SCB-2583] enable checkstyle trailing whitespace (#3097)
---
ci/checkstyle/checkstyle.xml | 5 +++++
.../core/parser/CompositeVertxRestAccessLogItemMeta.java | 2 +-
.../protobuf/internal/converter/SerializableParameterAdapter.java | 2 +-
core/src/main/java/org/apache/servicecomb/core/SCBEngine.java | 2 +-
.../apache/servicecomb/edge/core/EdgeAddHeaderClientFilter.java | 2 +-
.../servicecomb/foundation/common/utils/MimeTypesUtils.java | 8 ++++----
.../org/apache/servicecomb/foundation/common/utils/RSAUtils.java | 8 ++++----
.../src/test/resources/config/log4j.common.test.properties | 2 +-
.../org/apache/servicecomb/config/priority/PriorityProperty.java | 2 +-
.../src/main/java/io/protostuff/ByteArrayInputEx.java | 2 +-
.../servicecomb/registry/api/registry/MicroserviceFactory.java | 2 +-
.../apache/servicecomb/foundation/vertx/http/ReadStreamPart.java | 2 +-
.../vertx/http/VertxServerResponseToHttpServletResponse.java | 2 +-
.../authentication/consumer/RSAConsumerTokenManager.java | 2 +-
.../servicecomb/it/testcase/TestRestVertxTransportConfig.java | 2 +-
.../metrics/core/meter/invocation/AbstractInvocationMeters.java | 2 +-
.../provider/springmvc/reference/CseUriTemplateHandler.java | 2 +-
.../java/org/apache/servicecomb/serviceregistry/TestRegistry.java | 2 +-
.../consumer/TestSimpleMicroserviceInstancePing.java | 4 ++--
.../servicecomb/zeroconfig/AbstractZeroConfigRegistration.java | 2 +-
20 files changed, 31 insertions(+), 26 deletions(-)
diff --git a/ci/checkstyle/checkstyle.xml b/ci/checkstyle/checkstyle.xml
index 9684dc365..5beaa3105 100644
--- a/ci/checkstyle/checkstyle.xml
+++ b/ci/checkstyle/checkstyle.xml
@@ -31,6 +31,11 @@
</module>
<module name="NewlineAtEndOfFile">
</module>
+ <module name="RegexpSingleline">
+ <property name="format" value="\s+$"/>
+ <property name="message" value="Trailing whitespace"/>
+ <property name="severity" value="error"/>
+ </module>
<module name="TreeWalker">
<module name="AvoidStarImport"/>
<module name="EmptyStatement"/>
diff --git a/common/common-access-log/src/main/java/org/apache/servicecomb/common/accessLog/core/parser/CompositeVertxRestAccessLogItemMeta.java b/common/common-access-log/src/main/java/org/apache/servicecomb/common/accessLog/core/parser/CompositeVertxRestAccessLogItemMeta.java
index f36084797..6e4c477aa 100644
--- a/common/common-access-log/src/main/java/org/apache/servicecomb/common/accessLog/core/parser/CompositeVertxRestAccessLogItemMeta.java
+++ b/common/common-access-log/src/main/java/org/apache/servicecomb/common/accessLog/core/parser/CompositeVertxRestAccessLogItemMeta.java
@@ -22,7 +22,7 @@ import java.util.List;
/**
* Hold a group of {@link VertxRestAccessLogItemMeta} so that user can define
* only one VertxRestAccessLogItemMeta in spi loading file and load a group of meta.
- *
+ *
* Once the access log loading mechanism finds that a meta is CompositeVertxRestAccessLogItemMeta,
* the meta hold by it will be used in access log while this meta itself will be ignored.
*/
diff --git a/common/common-protobuf/src/main/java/org/apache/servicecomb/codec/protobuf/internal/converter/SerializableParameterAdapter.java b/common/common-protobuf/src/main/java/org/apache/servicecomb/codec/protobuf/internal/converter/SerializableParameterAdapter.java
index 66ba8bda3..fe2b8b811 100644
--- a/common/common-protobuf/src/main/java/org/apache/servicecomb/codec/protobuf/internal/converter/SerializableParameterAdapter.java
+++ b/common/common-protobuf/src/main/java/org/apache/servicecomb/codec/protobuf/internal/converter/SerializableParameterAdapter.java
@@ -38,7 +38,7 @@ public class SerializableParameterAdapter implements SwaggerTypeAdapter {
if ("array".equals(parameter.getType())) {
return parameter.getItems();
}
-
+
return null;
}
diff --git a/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java b/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
index 9ed9daa1c..10c443bbc 100644
--- a/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
+++ b/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
@@ -631,7 +631,7 @@ public class SCBEngine {
engine.setStatus(SCBStatus.UP);
engine.triggerEvent(EventType.AFTER_REGISTRY);
EventManager.unregister(this);
- // keep this message to be WARN, used to detect service ready.
+ // keep this message to be WARN, used to detect service ready.
LOGGER.warn("ServiceComb is ready.");
}
}
diff --git a/edge/edge-core/src/main/java/org/apache/servicecomb/edge/core/EdgeAddHeaderClientFilter.java b/edge/edge-core/src/main/java/org/apache/servicecomb/edge/core/EdgeAddHeaderClientFilter.java
index 95b67ddc3..c31539390 100644
--- a/edge/edge-core/src/main/java/org/apache/servicecomb/edge/core/EdgeAddHeaderClientFilter.java
+++ b/edge/edge-core/src/main/java/org/apache/servicecomb/edge/core/EdgeAddHeaderClientFilter.java
@@ -85,7 +85,7 @@ public class EdgeAddHeaderClientFilter implements HttpClientFilter {
if (!invocation.isEdge()) {
return;
}
-
+
HttpServletRequestEx oldRequest = invocation.getRequestEx();
publicHeaders.forEach(key -> {
String value = oldRequest.getHeader(key);
diff --git a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/MimeTypesUtils.java b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/MimeTypesUtils.java
index 4102b65cb..25bf4cb46 100644
--- a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/MimeTypesUtils.java
+++ b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/MimeTypesUtils.java
@@ -54,20 +54,20 @@ public class MimeTypesUtils {
}
public static List<String> getSortedAcceptableMimeTypes(String acceptHeader) {
- // accept anything when accept is not present
+ // accept anything when accept is not present
if (acceptHeader == null) {
return Collections.emptyList();
}
- // parse
+ // parse
String[] items = COMMA_SPLITTER.split(acceptHeader);
- // sort on quality
+ // sort on quality
Arrays.sort(items, ACCEPT_X_COMPARATOR);
List<String> list = new ArrayList<>(items.length);
for (String item : items) {
- // find any ; e.g.: "application/json;q=0.8"
+ // find any ; e.g.: "application/json;q=0.8"
int space = item.indexOf(';');
if (space != -1) {
diff --git a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
index 5923f9b5f..22742700b 100644
--- a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
+++ b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
@@ -74,7 +74,7 @@ public class RSAUtils {
}
/**
- * if has performance problem ,change Signature to ThreadLocal instance
+ * if has performance problem ,change Signature to ThreadLocal instance
*/
public static String sign(String content, PrivateKey privateKey)
throws NoSuchAlgorithmException, SignatureException, InvalidKeyException {
@@ -87,10 +87,10 @@ public class RSAUtils {
/**
*
- * if has performance problem ,change Signature to ThreadLocal instance
- * @param publicKey public key after base64 encode
+ * if has performance problem ,change Signature to ThreadLocal instance
+ * @param publicKey public key after base64 encode
* @param sign 签名
- * @param content original content
+ * @param content original content
* @return verify result
* @throws NoSuchAlgorithmException
* @throws InvalidKeySpecException
diff --git a/foundations/foundation-common/src/test/resources/config/log4j.common.test.properties b/foundations/foundation-common/src/test/resources/config/log4j.common.test.properties
index 0f636e4ba..1249e3835 100644
--- a/foundations/foundation-common/src/test/resources/config/log4j.common.test.properties
+++ b/foundations/foundation-common/src/test/resources/config/log4j.common.test.properties
@@ -15,7 +15,7 @@
# limitations under the License.
#
-# Override the log setting
+# Override the log setting
paas.logs.dir=./target/logs
paas.logs.file=cse.log
diff --git a/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/priority/PriorityProperty.java b/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/priority/PriorityProperty.java
index 3af93c84c..3d03267ae 100644
--- a/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/priority/PriorityProperty.java
+++ b/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/priority/PriorityProperty.java
@@ -192,7 +192,7 @@ public class PriorityProperty<T> {
return true;
}
}
-
+
return false;
}
}
diff --git a/foundations/foundation-protobuf/src/main/java/io/protostuff/ByteArrayInputEx.java b/foundations/foundation-protobuf/src/main/java/io/protostuff/ByteArrayInputEx.java
index 98a121173..f496a7094 100644
--- a/foundations/foundation-protobuf/src/main/java/io/protostuff/ByteArrayInputEx.java
+++ b/foundations/foundation-protobuf/src/main/java/io/protostuff/ByteArrayInputEx.java
@@ -3,7 +3,7 @@
//------------------------------------------------------------------------
//Licensed under the Apache License, Version 2.0 (the "License");
//you may not use this file except in compliance with the License.
-//You may obtain a copy of the License at
+//You may obtain a copy of the License at
//http://www.apache.org/licenses/LICENSE-2.0
//Unless required by applicable law or agreed to in writing, software
//distributed under the License is distributed on an "AS IS" BASIS,
diff --git a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/api/registry/MicroserviceFactory.java b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/api/registry/MicroserviceFactory.java
index e4bc81254..16b714728 100644
--- a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/api/registry/MicroserviceFactory.java
+++ b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/api/registry/MicroserviceFactory.java
@@ -86,7 +86,7 @@ public class MicroserviceFactory {
microservice.setFramework(createFramework());
microservice.setRegisterBy(CONFIG_DEFAULT_REGISTER_BY);
-
+
return microservice;
}
diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/ReadStreamPart.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/ReadStreamPart.java
index 09059de8b..54a917f78 100644
--- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/ReadStreamPart.java
+++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/ReadStreamPart.java
@@ -45,7 +45,7 @@ import io.vertx.core.streams.WriteStream;
* 1.getContentType<br>
* 2.getSubmittedFileName<br>
* extend from AbstractPart just because want to make it be Part type,
- * so that can be sent by
+ * so that can be sent by
* {@link org.apache.servicecomb.foundation.vertx.http.VertxServerResponseToHttpServletResponse#sendPart(Part) VertxServerResponseToHttpServletResponse.sendPart}
*/
public class ReadStreamPart extends AbstractPart {
diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
index 52a2c8864..668ce0022 100644
--- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
+++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
@@ -62,7 +62,7 @@ public class VertxServerResponseToHttpServletResponse extends AbstractHttpServle
public void setStatus(int sc) {
serverResponse.setStatusCode(sc);
}
-
+
@Override
public StatusType getStatusType() {
if (statusType == null) {
diff --git a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/consumer/RSAConsumerTokenManager.java b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/consumer/RSAConsumerTokenManager.java
index 4906d5e62..7304ec3a5 100644
--- a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/consumer/RSAConsumerTokenManager.java
+++ b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/consumer/RSAConsumerTokenManager.java
@@ -71,7 +71,7 @@ public class RSAConsumerTokenManager {
/**
* the TTL of Token is 24 hours
- * client token will expired 15 minutes early
+ * client token will expired 15 minutes early
*/
public boolean isExpired(RSAAuthenticationToken token) {
if (null == token) {
diff --git a/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestRestVertxTransportConfig.java b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestRestVertxTransportConfig.java
index 0e4fa4017..38cd8c0d2 100644
--- a/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestRestVertxTransportConfig.java
+++ b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestRestVertxTransportConfig.java
@@ -86,7 +86,7 @@ public class TestRestVertxTransportConfig {
consumers.getIntf().testClientReceiveHeaderSize(100001 - RESPONSE_HEADER.length());
Assertions.fail("an exception is expected!");
} catch (InvocationException e) {
- // in slow environment, may cause connection close.
+ // in slow environment, may cause connection close.
MatcherAssert.assertThat(ExceptionUtils.getExceptionMessageWithoutTrace(e),
CoreMatchers.anyOf(CoreMatchers.containsString("HTTP header is larger than 10000 bytes"),
CoreMatchers.containsString("Connection was closed")));
diff --git a/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/invocation/AbstractInvocationMeters.java b/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/invocation/AbstractInvocationMeters.java
index a82560907..9491120d3 100644
--- a/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/invocation/AbstractInvocationMeters.java
+++ b/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/invocation/AbstractInvocationMeters.java
@@ -33,7 +33,7 @@ public abstract class AbstractInvocationMeters {
private final Map<String, AbstractInvocationMeter> metersMap = new ConcurrentHashMapEx<>();
- // not care for concurrency, just for make build key faster
+ // not care for concurrency, just for make build key faster
private int maxKeyLen = 64;
public AbstractInvocationMeters(Registry registry) {
diff --git a/providers/provider-springmvc/src/main/java/org/apache/servicecomb/provider/springmvc/reference/CseUriTemplateHandler.java b/providers/provider-springmvc/src/main/java/org/apache/servicecomb/provider/springmvc/reference/CseUriTemplateHandler.java
index ea5a85820..ecc5ff788 100644
--- a/providers/provider-springmvc/src/main/java/org/apache/servicecomb/provider/springmvc/reference/CseUriTemplateHandler.java
+++ b/providers/provider-springmvc/src/main/java/org/apache/servicecomb/provider/springmvc/reference/CseUriTemplateHandler.java
@@ -134,7 +134,7 @@ public class CseUriTemplateHandler extends org.springframework.web.util.DefaultU
throw new IllegalArgumentException("[" + uri + "] is not a valid URI");
}
}
-
+
private URI createUri(String uriTemplate, UriComponents uriComponents) {
String strUri = uriComponents.toUriString();
diff --git a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/TestRegistry.java b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/TestRegistry.java
index 9f0d4f6b3..9a9338110 100644
--- a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/TestRegistry.java
+++ b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/TestRegistry.java
@@ -114,7 +114,7 @@ public class TestRegistry {
Assertions.assertEquals(properties, RegistryUtils.getMicroserviceInstance().getProperties());
Assertions.assertEquals(microservice, RegistryUtils.getMicroservice(microservice.getServiceId()));
-
+
RegistryUtils.updateMicroserviceProperties(properties);
Assertions.assertEquals(properties, RegistryUtils.getMicroservice().getProperties());
diff --git a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/consumer/TestSimpleMicroserviceInstancePing.java b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/consumer/TestSimpleMicroserviceInstancePing.java
index 086c65863..04ff785c4 100644
--- a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/consumer/TestSimpleMicroserviceInstancePing.java
+++ b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/consumer/TestSimpleMicroserviceInstancePing.java
@@ -44,7 +44,7 @@ public class TestSimpleMicroserviceInstancePing {
ss.close();
Assertions.assertFalse(ping.ping(instance));
}
-
+
@Test
public void testPing_more_endpoin() throws IOException {
SimpleMicroserviceInstancePing ping = new SimpleMicroserviceInstancePing();
@@ -58,5 +58,5 @@ public class TestSimpleMicroserviceInstancePing {
ss.close();
Assertions.assertFalse(ping.ping(instance));
}
-
+
}
diff --git a/service-registry/registry-zero-config/src/main/java/org/apache/servicecomb/zeroconfig/AbstractZeroConfigRegistration.java b/service-registry/registry-zero-config/src/main/java/org/apache/servicecomb/zeroconfig/AbstractZeroConfigRegistration.java
index 39f0c00b1..b69d98af6 100644
--- a/service-registry/registry-zero-config/src/main/java/org/apache/servicecomb/zeroconfig/AbstractZeroConfigRegistration.java
+++ b/service-registry/registry-zero-config/src/main/java/org/apache/servicecomb/zeroconfig/AbstractZeroConfigRegistration.java
@@ -44,7 +44,7 @@ public abstract class AbstractZeroConfigRegistration extends AbstractLightweight
@Override
public void run() {
super.run();
-
+
postRegisteredEvent();
startRegister(config.getHeartbeatInterval());
}