(servicecomb-java-chassis) branch master updated: fix: fix loaded variable not assigned (#4065)

[shoothzj]

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 d5f5714d1 fix: fix loaded variable not assigned (#4065)
d5f5714d1 is described below

commit d5f5714d1afac5675a49996e0ccd884fb617041c
Author: ZhangJian He <shoot...@gmail.com>
AuthorDate: Thu Nov 30 13:52:48 2023 +0800

    fix: fix loaded variable not assigned (#4065)
    
    Signed-off-by: ZhangJian He <shoot...@gmail.com>
---
 ci/spotbugs/exclude.xml                                              | 5 -----
 .../servicecomb/huaweicloud/servicestage/AKSKAuthHeaderProvider.java | 3 ++-
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/ci/spotbugs/exclude.xml b/ci/spotbugs/exclude.xml
index fc38bcbc3..453eea4a1 100644
--- a/ci/spotbugs/exclude.xml
+++ b/ci/spotbugs/exclude.xml
@@ -377,11 +377,6 @@
         <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/huawei-cloud/servicestage/src/main/java/org/apache/servicecomb/huaweicloud/servicestage/AKSKAuthHeaderProvider.java
 
b/huawei-cloud/servicestage/src/main/java/org/apache/servicecomb/huaweicloud/servicestage/AKSKAuthHeaderProvider.java
index 3469df6a7..e192a4fbd 100644
--- 
a/huawei-cloud/servicestage/src/main/java/org/apache/servicecomb/huaweicloud/servicestage/AKSKAuthHeaderProvider.java
+++ 
b/huawei-cloud/servicestage/src/main/java/org/apache/servicecomb/huaweicloud/servicestage/AKSKAuthHeaderProvider.java
@@ -65,7 +65,7 @@ public class AKSKAuthHeaderProvider implements 
AuthHeaderProvider {
 
   private boolean enabled;
 
-  private final boolean loaded = false;
+  private boolean loaded = false;
 
   private final Environment environment;
 
@@ -98,6 +98,7 @@ public class AKSKAuthHeaderProvider implements 
AuthHeaderProvider {
       headers.put(X_SERVICE_AK, getAccessKey());
       headers.put(X_SERVICE_SHAAKSK, getSecretKey());
       headers.put(X_SERVICE_PROJECT, getProject());
+      loaded = true;
     }
   }