improve coding styles

JacksonTian · JacksonTian · commit 0d6dc9e5a9d4 · 2024-07-09T17:57:41.000+08:00
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/AcsRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/AcsRequest.java
@@ -24,7 +24,7 @@ public abstract class AcsRequest<T extends AcsResponse> extends HttpRequest {
     private UserAgentConfig userAgentConfig;
     protected ISignatureComposer composer = null;
     private String version = null;
-    private String product = null;
+    private String product;
     private String actionName = null;
     private String regionId = null;
     private String securityToken = null;
@@ -76,7 +76,7 @@ public static String concatQueryString(Map<String, String> parameters) throws Un
             return null;
         }
 
-        StringBuilder urlBuilder = new StringBuilder("");
+        StringBuilder urlBuilder = new StringBuilder();
         for (Entry<String, String> entry : parameters.entrySet()) {
             String key = entry.getKey();
             String val = entry.getValue();
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/CommonRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/CommonRequest.java
@@ -509,30 +509,6 @@ public Map<String, String> getSysPathParameters() {
         return pathParameters;
     }
 
-    public void setSysAccept(FormatType type) {
-        this.accept = type;
-    }
-
-    public FormatType getSysAccept() {
-        return this.accept;
-    }
-
-    public SignatureVersion getSysSignatureVersion() {
-        return signatureVersion;
-    }
-
-    public void setSysSignatureVersion(SignatureVersion signatureVersion) {
-        this.signatureVersion = signatureVersion;
-    }
-
-    public SignatureAlgorithm getSysSignatureAlgorithm() {
-        return signatureAlgorithm;
-    }
-
-    public void setSysSignatureAlgorithm(SignatureAlgorithm signatureAlgorithm) {
-        this.signatureAlgorithm = signatureAlgorithm;
-    }
-
     public RetryPolicy getSysRetryPolicy() {
         return this.retryPolicy;
     }
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/RoaAcsRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/RoaAcsRequest.java
@@ -114,7 +114,7 @@ protected void putPathParameter(String name, String value) {
     @Override
     public String composeUrl(String endpoint, Map<String, String> queries) throws UnsupportedEncodingException {
         Map<String, String> mapQueries = (queries == null) ? this.getSysQueryParameters() : queries;
-        StringBuilder urlBuilder = new StringBuilder("");
+        StringBuilder urlBuilder = new StringBuilder();
         urlBuilder.append(this.getSysProtocol().toString());
         urlBuilder.append("://").append(endpoint);
         if (null != this.uriPattern) {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/RpcAcsRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/RpcAcsRequest.java
@@ -131,7 +131,7 @@ public void setSysSecurityToken(String securityToken) {
     @Override
     public String composeUrl(String endpoint, Map<String, String> queries) throws UnsupportedEncodingException {
         Map<String, String> mapQueries = (queries == null) ? this.getSysQueryParameters() : queries;
-        StringBuilder urlBuilder = new StringBuilder("");
+        StringBuilder urlBuilder = new StringBuilder();
         urlBuilder.append(this.getSysProtocol().toString());
         urlBuilder.append("://").append(endpoint);
         if (-1 == urlBuilder.indexOf("?")) {
@@ -166,7 +166,7 @@ public HttpRequest signRequest(Signer signer, AlibabaCloudCredentials credential
                 format);
         headerMap.putAll(this.getSysHeaders());
         headerMap = this.composer.refreshSignParameters(headerMap, signer, null, null);
-        if (imutableMap.get("RegionId") == null && this.getSysRegionId() != null && !this.getSysRegionId().equals("")) {
+        if (imutableMap.get("RegionId") == null && this.getSysRegionId() != null && !this.getSysRegionId().isEmpty()) {
             if ((bodyParams == null || bodyParams.get("RegionId") == null)) {
                 imutableMap.put("RegionId", getSysRegionId());
             }
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/DefaultCredentialsProvider.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/DefaultCredentialsProvider.java
@@ -27,7 +27,7 @@ public DefaultCredentialsProvider() throws ClientException {
         defaultProviders.add(new ProfileCredentialsProvider());
         String roleName = AuthUtils.getEnvironmentECSMetaData();
         if (roleName != null) {
-            if (roleName.length() == 0) {
+            if (roleName.isEmpty()) {
                 throw new ClientException("Environment variable roleName('ALIBABA_CLOUD_ECS_METADATA') cannot be empty");
             }
             defaultProviders.add(new InstanceProfileCredentialsProvider(roleName));
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/LegacyCredentials.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/LegacyCredentials.java
@@ -5,8 +5,8 @@ public class LegacyCredentials implements AlibabaCloudCredentials {
 
     private final Credential legacyCredential;
 
-    public LegacyCredentials(Credential legacyCrendential) {
-        this.legacyCredential = legacyCrendential;
+    public LegacyCredentials(Credential legacyCredential) {
+        this.legacyCredential = legacyCredential;
     }
 
     @Override
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/ProfileCredentialsProvider.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/ProfileCredentialsProvider.java
@@ -31,7 +31,7 @@ public AlibabaCloudCredentials getCredentials() throws ClientException {
         if (filePath == null) {
             filePath = AuthConstant.DEFAULT_CREDENTIALS_FILE_PATH;
         }
-        if (filePath.length() == 0) {
+        if (filePath.isEmpty()) {
             throw new ClientException("The specified credentials file is empty");
         }
         Wini ini;
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/SHA256withRSASigner.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/SHA256withRSASigner.java
@@ -41,8 +41,7 @@ public String signString(String stringToSign, String accessKeySecret) {
             rsaSign.initSign(privateKey);
             rsaSign.update(stringToSign.getBytes(ENCODING));
             byte[] sign = rsaSign.sign();
-            String signature = DatatypeConverter.printBase64Binary(sign);
-            return signature;
+            return DatatypeConverter.printBase64Binary(sign);
         } catch (NoSuchAlgorithmException e) {
             throw new IllegalArgumentException(e.toString());
         } catch (InvalidKeySpecException e) {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/STSGetSessionAccessKeyCredentialsProvider.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/STSGetSessionAccessKeyCredentialsProvider.java
@@ -65,7 +65,7 @@ private BasicSessionCredentials getNewSessionCredentials() throws ClientExceptio
 
         return new BasicSessionCredentials(
                 response.getSessionAccessKey().getSessionAccessKeyId(),
-                response.getSessionAccessKey().getSessionAccessKeySecert(),
+                response.getSessionAccessKey().getSessionAccessKeySecret(),
                 null,
                 sessionDurationSeconds
         );
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/Signer.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/Signer.java
@@ -33,16 +33,6 @@ public abstract class Signer {
     private final static Signer HMAC_SM3 = new HmacSM3Signer();
     private final static Signer HMAC_SHA256 = new HmacSHA256Signer();
 
-    public static Signer getSigner(AlibabaCloudCredentials credentials) {
-        if (credentials instanceof KeyPairCredentials) {
-            return SHA256_WITH_RSA_SIGNER;
-        } else if (credentials instanceof BearerTokenCredentials) {
-            return BEARER_TOKEN_SIGNER;
-        } else {
-            return HMACSHA1_SIGNER;
-        }
-    }
-
     public static Signer getSigner(AlibabaCloudCredentials credentials, SignatureVersion signatureVersion, SignatureAlgorithm signatureAlgorithm) {
         switch (signatureVersion) {
             case V3:
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/V3SignatureComposer.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/V3SignatureComposer.java
@@ -84,7 +84,7 @@ private Map<String, String> getCanonicalizedHeadersMap(Map<String, String> heade
     }
 
     private String getCanonicalizedResource(Map<String, String> query) {
-        if (query == null || query.size() == 0) {
+        if (query == null || query.isEmpty()) {
             return "";
         }
         String[] keys = query.keySet().toArray(new String[query.size()]);
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/AssumeRoleRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/AssumeRoleRequest.java
@@ -19,16 +19,9 @@
 
 package com.aliyuncs.auth.sts;
 
-/**
- * Created by haowei.yao on 2017/9/14.
- */
-
 import com.aliyuncs.RpcAcsRequest;
 import com.aliyuncs.http.ProtocolType;
 
-/**
- * @author auto create
- */
 public class AssumeRoleRequest extends RpcAcsRequest<AssumeRoleResponse> {
 
     private Long durationSeconds;
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/GenerateSessionAccessKeyResponse.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/GenerateSessionAccessKeyResponse.java
@@ -59,13 +59,13 @@ public void setSessionAccessKeyId(String sessionAccessKeyId) {
             this.sessionAccessKeyId = sessionAccessKeyId;
         }
 
-        public String getSessionAccessKeySecert() {
+        public String getSessionAccessKeySecret() {
             return sessionAccessKeySecret;
         }
 
         @XmlElement(name = "SessionAccessKeySecret")
-        public void setSessionAccessKeySecert(String sessionAccessKeySecert) {
-            this.sessionAccessKeySecret = sessionAccessKeySecert;
+        public void setSessionAccessKeySecret(String sessionAccessKeySecret) {
+            this.sessionAccessKeySecret = sessionAccessKeySecret;
         }
 
         public String getExpiration() {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/GetSessionAccessKeyResponseUnmarshaller.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/auth/sts/GetSessionAccessKeyResponseUnmarshaller.java
@@ -12,7 +12,7 @@ public static GenerateSessionAccessKeyResponse unmarshall(GenerateSessionAccessK
 
         SessionAccessKey credentials = new SessionAccessKey();
         credentials.setSessionAccessKeyId(context.stringValue("GenerateSessionAccessKeyResponse.SessionAccessKey.SessionAccessKeyId"));
-        credentials.setSessionAccessKeySecert(context.stringValue("GenerateSessionAccessKeyResponse.SessionAccessKey.SessionAccessKeySecret"));
+        credentials.setSessionAccessKeySecret(context.stringValue("GenerateSessionAccessKeyResponse.SessionAccessKey.SessionAccessKeySecret"));
         credentials.setExpiration(context.stringValue("GenerateSessionAccessKeyResponse.SessionAccessKey.Expiration"));
 
         getSessionAccessKeyResponse.setSessionAccessKey(credentials);
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/EndpointResolverRules.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/EndpointResolverRules.java
@@ -50,14 +50,14 @@ private String getEndpoint(String productCode, String regionId) {
                 }
             }
 
-            String endpoint = "";
+            String endpoint;
             if ("regional".equals(this.productEndpointRegional)) {
                 endpoint = "<product_id><suffix><network>.<region_id>.aliyuncs.com";
                 endpoint = endpoint.replace("<region_id>", regionId.toLowerCase());
             } else {
                 endpoint = "<product_id><suffix><network>.aliyuncs.com";
             }
-            if (this.productSuffix == null || "".equals(this.productSuffix)) {
+            if (StringUtils.isEmpty(this.productSuffix)) {
                 endpoint = endpoint.replace("<suffix>", "");
             } else {
                 endpoint = endpoint.replace("<suffix>", "-" + this.productSuffix.toLowerCase());
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/LocationServiceEndpointResolver.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/LocationServiceEndpointResolver.java
@@ -6,6 +6,7 @@
 import com.aliyuncs.exceptions.ClientException;
 import com.aliyuncs.http.FormatType;
 import com.aliyuncs.http.ProtocolType;
+import com.aliyuncs.utils.StringUtils;
 
 import java.util.HashSet;
 import java.util.Set;
@@ -38,7 +39,7 @@ public static void setLocationServiceEndpoint(String endpoint) {
 
     @Override
     public String resolve(ResolveEndpointRequest request) throws ClientException {
-        if (request.locationServiceCode == null || request.locationServiceCode.length() == 0) {
+        if (StringUtils.isEmpty(request.locationServiceCode)) {
             return null;
         }
 
@@ -106,7 +107,6 @@ private void callLocationService(String key, ResolveEndpointRequest request) thr
         validProductCodes.add(request.productCodeLower);
         validRegionIds.add(request.regionId);
 
-
         boolean foundFlag = false;
         for (DescribeEndpointsResponse.Endpoint endpoint : response.getEndpoints()) {
             if (endpoint.getSerivceCode().equals(request.locationServiceCode)
@@ -143,10 +143,6 @@ public String makeEndpointKey(ResolveEndpointRequest request) {
         );
     }
 
-    public String makeRegionIdKey(ResolveEndpointRequest request) {
-        return request.locationServiceCode + "." + request.regionId + "." + request.endpointType;
-    }
-
     public String makeEndpointKey(String productCode, String locationServiceCode, String regionId,
                                   String endpointType) {
         return productCode.toLowerCase() + "." + locationServiceCode + "."
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/ResolveEndpointRequest.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/endpoint/ResolveEndpointRequest.java
@@ -1,17 +1,19 @@
 package com.aliyuncs.endpoint;
 
+import com.aliyuncs.utils.StringUtils;
+
 import java.util.HashMap;
 
 public class ResolveEndpointRequest {
 
     public static final String ENDPOINT_TYPE_INNER = "innerAPI";
     public static final String ENDPOINT_TYPE_OPEN = "openAPI";
 
-    public String productCode = null;
-    public String regionId = null;
-    public String endpointType = null;
-    public String locationServiceCode = null;
-    public String productCodeLower = null;
+    public String productCode;
+    public String regionId;
+    public String endpointType;
+    public String locationServiceCode;
+    public String productCodeLower;
     public HashMap<String, String> productEndpointMap = null;
     public String productEndpointRegional = null;
     public String productNetwork = "public";
@@ -24,7 +26,7 @@ public ResolveEndpointRequest(String regionId, String productCode,
         this.productCode = productCode;
         this.productCodeLower = productCode.toLowerCase();
 
-        if (endpointType == null || endpointType.length() == 0) {
+        if (StringUtils.isEmpty(endpointType)) {
             endpointType = ENDPOINT_TYPE_OPEN;
         }
 
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/CallBack.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/CallBack.java
@@ -2,9 +2,6 @@
 
 /**
  * 用于异步调用时的回调逻辑
- *
- * @author VK.Gao
- * @date 2017/03/02
  */
 public interface CallBack {
 
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/CompositeX509TrustManager.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/CompositeX509TrustManager.java
@@ -27,7 +27,7 @@ public CompositeX509TrustManager(List<X509TrustManager> trustManagers) {
     }
 
     @Override
-    public void checkClientTrusted(X509Certificate[] chain, String authType) throws CertificateException {
+    public void checkClientTrusted(X509Certificate[] chain, String authType) {
         // do nothing
     }
 
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/HttpClientFactory.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/HttpClientFactory.java
@@ -20,7 +20,7 @@ public static IHttpClient buildClient(IClientProfile profile) {
                 profile.setHttpClientConfig(clientConfig);
             }
 
-            String customClientClassName = null;
+            String customClientClassName;
             if (clientConfig.isCompatibleMode()) {
                 customClientClassName = COMPATIBLE_HTTP_CLIENT_CLASS_NAME;
             } else if (clientConfig.getClientType() == HttpClientType.Custom && !StringUtils.isEmpty(clientConfig.getCustomClientClassName())) {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/HttpUtil.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/HttpUtil.java
@@ -53,7 +53,7 @@ public static String debugHttpRequest(HttpRequest request) {
             StringBuilder debugString = new StringBuilder();
 
             String sysUrl = request.getSysUrl();
-            URL url = null;
+            URL url;
             try {
                 url = new URL(sysUrl);
                 debugString.append("> " + request.getSysMethod() + " " + url.getProtocol().toUpperCase() + "/1.1\n> ");
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/IHttpClient.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/IHttpClient.java
@@ -16,7 +16,6 @@ public IHttpClient(HttpClientConfig clientConfig) throws ClientException {
         }
         this.clientConfig = clientConfig;
         init(clientConfig);
-
     }
 
     public IHttpClient() {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/clients/ApacheHttpClient.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/clients/ApacheHttpClient.java
@@ -120,8 +120,7 @@ private SSLConnectionSocketFactory createSSLConnectionSocketFactory() throws Cli
                 } else {
                     hostnameVerifier = new DefaultHostnameVerifier();
                 }
-                SSLConnectionSocketFactory sslConnectionSocketFactory = new SSLConnectionSocketFactory(sslContext, hostnameVerifier);
-                return sslConnectionSocketFactory;
+                return new SSLConnectionSocketFactory(sslContext, hostnameVerifier);
             } else {
                 HostnameVerifier hostnameVerifier;
                 if (null == clientConfig.getHostnameVerifier()) {
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/clients/CompatibleUrlConnClient.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/http/clients/CompatibleUrlConnClient.java
@@ -48,9 +48,9 @@ protected void init(HttpClientConfig clientConfig) {
     @Override
     public HttpResponse syncInvoke(HttpRequest request) throws IOException, ClientException {
         InputStream content = null;
-        HttpResponse response = null;
+        HttpResponse response;
         HttpURLConnection httpConn = buildHttpConnection(request);
-        OutputStream out = null;
+        OutputStream out;
 
         try {
             httpConn.connect();
@@ -85,8 +85,7 @@ public Future<HttpResponse> asyncInvoke(HttpRequest apiRequest, CallBack callbac
     }
 
     private boolean calcIgnoreSSLCert(HttpRequest request) {
-        boolean ignoreSSLCert = request.isIgnoreSSLCerts() ? request.isIgnoreSSLCerts() : clientConfig.isIgnoreSSLCerts();
-        return ignoreSSLCert;
+        return request.isIgnoreSSLCerts() ? request.isIgnoreSSLCerts() : clientConfig.isIgnoreSSLCerts();
     }
 
     private CompositeX509TrustManager calcX509TrustManager(HttpRequest request) throws KeyStoreException, NoSuchAlgorithmException {
@@ -246,7 +245,7 @@ private void setConnectionRequestProperty(HttpURLConnection httpConn, HttpReques
     private HttpURLConnection buildHttpConnection(HttpRequest request) throws IOException, ClientException {
         checkHttpRequest(request);
         String strUrl = request.getSysUrl();
-        URL url = null;
+        URL url;
         String[] urlArray = null;
         if (MethodType.POST.equals(request.getSysMethod()) && null == request.getHttpContent()) {
             urlArray = strUrl.split("\\?");
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/profile/DefaultProfile.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/profile/DefaultProfile.java
@@ -2,7 +2,6 @@
 
 import com.aliyuncs.auth.*;
 import com.aliyuncs.endpoint.DefaultEndpointResolver;
-import com.aliyuncs.exceptions.ClientException;
 import com.aliyuncs.http.FormatType;
 import com.aliyuncs.http.HttpClientConfig;
 import com.aliyuncs.utils.ParameterHelper;
@@ -55,8 +54,7 @@ public synchronized static DefaultProfile getProfile(String regionId, ICredentia
     }
 
     public synchronized static DefaultProfile getProfile(String regionId, String accessKeyId, String secret) {
-        Credential creden = new Credential(accessKeyId, secret);
-        profile = new DefaultProfile(regionId, creden);
+        profile = new DefaultProfile(regionId, new Credential(accessKeyId, secret));
         return profile;
     }
 
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/profile/IClientProfile.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/profile/IClientProfile.java
@@ -7,7 +7,6 @@
 import com.aliyuncs.http.HttpClientConfig;
 import org.slf4j.Logger;
 
-@SuppressWarnings("deprecation")
 public interface IClientProfile {
 
     /**
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/reader/XmlReader.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/reader/XmlReader.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/transform/UnmarshallerContext.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/transform/UnmarshallerContext.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/unmarshaller/Unmarshaller.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/unmarshaller/Unmarshaller.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/AuthUtils.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/AuthUtils.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/Base64Helper.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/Base64Helper.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/MapUtils.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/MapUtils.java
diff --git a/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/XmlUtils.java b/aliyun-java-sdk-core/src/main/java/com/aliyuncs/utils/XmlUtils.java
diff --git a/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/STSGetSessionAccessKeyCredentialsProviderTest.java b/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/STSGetSessionAccessKeyCredentialsProviderTest.java
diff --git a/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/sts/GenerateSessionAccessKeyResponseTest.java b/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/sts/GenerateSessionAccessKeyResponseTest.java

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public DefaultCredentialsProvider() throws ClientException {`
`27`	`27`	`defaultProviders.add(new ProfileCredentialsProvider());`
`28`	`28`	`String roleName = AuthUtils.getEnvironmentECSMetaData();`
`29`	`29`	`if (roleName != null) {`
`30`		`- if (roleName.length() == 0) {`
	`30`	`+ if (roleName.isEmpty()) {`
`31`	`31`	`throw new ClientException("Environment variable roleName('ALIBABA_CLOUD_ECS_METADATA') cannot be empty");`
`32`	`32`	`}`
`33`	`33`	`defaultProviders.add(new InstanceProfileCredentialsProvider(roleName));`
Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,8 @@ public class LegacyCredentials implements AlibabaCloudCredentials {`
`5`	`5`
`6`	`6`	`private final Credential legacyCredential;`
`7`	`7`
`8`		`- public LegacyCredentials(Credential legacyCrendential) {`
`9`		`- this.legacyCredential = legacyCrendential;`
	`8`	`+ public LegacyCredentials(Credential legacyCredential) {`
	`9`	`+ this.legacyCredential = legacyCredential;`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public AlibabaCloudCredentials getCredentials() throws ClientException {`
`31`	`31`	`if (filePath == null) {`
`32`	`32`	`filePath = AuthConstant.DEFAULT_CREDENTIALS_FILE_PATH;`
`33`	`33`	`}`
`34`		`- if (filePath.length() == 0) {`
	`34`	`+ if (filePath.isEmpty()) {`
`35`	`35`	`throw new ClientException("The specified credentials file is empty");`
`36`	`36`	`}`
`37`	`37`	`Wini ini;`