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 f44383d8b com.github.spotbugs:spotbugs-maven-plugin from 4.7.3.5 to
4.8.1.0 (#4064)
f44383d8b is described below
commit f44383d8bded22d98c243d1174c954f38ab18df8
Author: Bristo <[email protected]>
AuthorDate: Thu Nov 30 07:41:30 2023 +0800
com.github.spotbugs:spotbugs-maven-plugin from 4.7.3.5 to 4.8.1.0 (#4064)
Co-authored-by: ZhangJian He <[email protected]>
---
ci/spotbugs/exclude.xml | 155 +++++++++++++++++++++
.../springboot/pojo/client/PojoClientTest.java | 2 +-
pom.xml | 2 +-
3 files changed, 157 insertions(+), 2 deletions(-)
diff --git a/ci/spotbugs/exclude.xml b/ci/spotbugs/exclude.xml
index b59b0b113..fc38bcbc3 100644
--- a/ci/spotbugs/exclude.xml
+++ b/ci/spotbugs/exclude.xml
@@ -35,6 +35,121 @@
<Bug pattern="BC_VACUOUS_INSTANCEOF"/>
</Match>
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class name="org.apache.servicecomb.foundation.common.Version"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.common.base.DynamicEnumCache"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.common.event.SimpleSubscriber"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.common.http.HttpStatusManager"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.common.net.URIEndpointObject"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.config.file.MicroserviceConfigLoader"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.vertx.server.TcpParser"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.swagger.generator.core.model.SwaggerOperations"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.core.executor.ThreadPoolExecutorEx"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.protobuf.internal.schema.EnumMeta"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.protobuf.internal.schema.deserializer.repeated.impl.enums.EnumNotPackedReadSchemas$EnumNotPackedReaders"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.protobuf.internal.schema.deserializer.repeated.impl.enums.EnumPackedReadSchemas$EnumPackedReaders"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.protobuf.internal.schema.serializer.repeated.impl.enums.EnumNotPackedWriteSchemas$EnumNotPackedWriters"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.foundation.protobuf.internal.schema.serializer.repeated.impl.enums.EnumPackedWriteSchemas$EnumsPackedWriters"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.localregistry.LocalDiscoveryInstance"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.common.rest.definition.path.PathRegExp"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.common.accessLog.core.element.impl.ConfigurableDatetimeAccessItem"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.common.accessLog.core.parser.impl.VertxRestAccessLogPatternParser"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.loadbalance.filter.PriorityInstancePropertyDiscoveryFilter$PriorityInstanceProperty"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class name="org.apache.servicecomb.http.client.common.URLEndPoint"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class
name="org.apache.servicecomb.http.client.common.WebSocketTransport"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class name="org.apache.servicecomb.zeroconfig.multicast.Multicast"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="CT_CONSTRUCTOR_THROW"/>
+ <Class name="org.apache.servicecomb.demo.edge.business.ImplV2"/>
+ </Match>
+
<Match>
<Bug pattern="DC_DOUBLECHECK"/>
</Match>
@@ -186,6 +301,41 @@
<Bug pattern="OBL_UNSATISFIED_OBLIGATION"/>
</Match>
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class
name="org.apache.servicecomb.demo.localRegistryServer.SelfServiceInvoker"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class name="org.apache.servicecomb.demo.mapnull.ParseRequest"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class name="org.apache.servicecomb.demo.mapnull.ParseResponse"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class name="org.apache.servicecomb.demo.pojo.client.PojoClient"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class name="org.apache.servicecomb.demo.registry.SelfServiceInvoker"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class
name="org.apache.servicecomb.demo.zeroconfig.server.SelfServiceInvoker"/>
+ </Match>
+
+ <Match>
+ <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
+ <Class
name="org.apache.servicecomb.demo.zeroconfig.edge.SelfServiceInvoker"/>
+ </Match>
+
<Match>
<Bug pattern="RC_REF_COMPARISON"/>
</Match>
@@ -227,6 +377,11 @@
<Bug pattern="SSD_DO_NOT_USE_INSTANCE_LOCK_ON_SHARED_STATIC_DATA"/>
</Match>
+ <Match>
+ <Bug pattern="SS_SHOULD_BE_STATIC"/>
+ <Class
name="org.apache.servicecomb.huaweicloud.servicestage.AKSKAuthHeaderProvider"/>
+ </Match>
+
<Match>
<Bug pattern="THROWS_METHOD_THROWS_CLAUSE_BASIC_EXCEPTION"/>
</Match>
diff --git
a/demo/demo-spring-boot-transport/demo-spring-boot-pojo-client/src/main/java/org/apache/servicecomb/demo/springboot/pojo/client/PojoClientTest.java
b/demo/demo-spring-boot-transport/demo-spring-boot-pojo-client/src/main/java/org/apache/servicecomb/demo/springboot/pojo/client/PojoClientTest.java
index e087b0b62..4e0525a12 100644
---
a/demo/demo-spring-boot-transport/demo-spring-boot-pojo-client/src/main/java/org/apache/servicecomb/demo/springboot/pojo/client/PojoClientTest.java
+++
b/demo/demo-spring-boot-transport/demo-spring-boot-pojo-client/src/main/java/org/apache/servicecomb/demo/springboot/pojo/client/PojoClientTest.java
@@ -38,7 +38,7 @@ public class PojoClientTest {
@RpcReference(microserviceName = "spring-boot-pojo-server", schemaId =
"server")
public static Test test;
- public static Test testFromXml;
+ private static Test testFromXml;
public static final byte[] buffer = new byte[1024];
diff --git a/pom.xml b/pom.xml
index 5381395bb..eccbf22dd 100644
--- a/pom.xml
+++ b/pom.xml
@@ -66,7 +66,7 @@
<release-maven-plugin.version>2.5.3</release-maven-plugin.version>
<site-maven-plugin.version>3.12.1</site-maven-plugin.version>
<source-maven-plugin.version>3.2.1</source-maven-plugin.version>
- <spotbug-maven-plugin.version>4.7.3.5</spotbug-maven-plugin.version>
+ <spotbug-maven-plugin.version>4.8.1.0</spotbug-maven-plugin.version>
<spring-boot-maven-plugin.version>3.1.3</spring-boot-maven-plugin.version>
<surefire-maven-plugin.version>3.2.2</surefire-maven-plugin.version>
<!-- plugin version end -->
