fix comments

Author: baiyubin2020

src/main/java/com/aliyun/oss/common/auth/CredentialsProviderFactory.java

@@ -55,7 +55,7 @@ public static DefaultCredentialProvider newDefaultCredentialProvider(String acce
      */
     public DefaultCredentialProvider newDefaultCredentialProvider(String accessKeyId, String secretAccessKey,
             String securityToken) {
-        return new DefaultCredentialProvider(accessKeyId, secretAccessKey, null);
+        return new DefaultCredentialProvider(accessKeyId, secretAccessKey, securityToken);
     }
 
     /**

src/main/java/com/aliyun/oss/common/auth/ProfileConfigFile.java

@@ -122,10 +122,10 @@ public void refresh() {
             String secretAccessKey = StringUtils.trim(profileProperties.get(AuthUtils.OSS_SECRET_ACCESS_KEY));
             String sessionToken = StringUtils.trim(profileProperties.get(AuthUtils.OSS_SESSION_TOKEN));
 
-            if (accessKeyId == null || accessKeyId.equals("")) {
+            if (StringUtils.isNullOrEmpty(accessKeyId)) {
                 throw new InvalidCredentialsException("Access key id should not be null or empty.");
             }
-            if (secretAccessKey == null || secretAccessKey.equals("")) {
+            if (StringUtils.isNullOrEmpty(secretAccessKey)) {
                 throw new InvalidCredentialsException("Secret access key should not be null or empty.");
             }
 

src/main/java/com/aliyun/oss/common/utils/IniEditor.java

@@ -577,9 +577,15 @@ public void load(String filename) throws IOException {
      *             at an I/O problem
      */
     public void load(File file) throws IOException {
-        InputStream in = new FileInputStream(file);
-        load(in);
-        in.close();
+        InputStream in = null;
+        try {
+        	in = new FileInputStream(file);
+        	load(in);
+        } finally {
+        	if (in != null) {
+        		in.close();
+        	}
+        }
     }
 
     /**

src/test/java/com/aliyun/oss/common/provider/InstanceProfileCredentialsTest.java

@@ -64,7 +64,7 @@ public void testWillSoonExpire() {
     public void testExpiredFactor() {
         try {
             InstanceProfileCredentials credentials = new InstanceProfileCredentials(ACCESS_KEY_ID, ACCESS_KEY_SECRET, null,
-                    "2020-11-11T11:11:11Z").withExpiredFactor(10000000.0);
+                    "2010-11-11T11:11:11Z").withExpiredFactor(10.0);
             Thread.sleep(1000);
             Assert.assertTrue(credentials.willSoonExpire());
 

src/test/java/com/aliyun/oss/common/provider/ProfilesConfigFileTest.java

@@ -124,6 +124,9 @@ public void testGetCredentialsWithLoader() {
     @Test
     public void testGetFreshCredentials() {
         try {
+        	new File(AuthUtils.DEFAULT_PROFILE_PATH).delete();
+        	Thread.sleep(1000);
+        	
             Map<String, String> options = new HashMap<String, String>();
             options.put(AuthUtils.OSS_ACCESS_KEY_ID, TEST_ACCESS_KEY_ID);
             options.put(AuthUtils.OSS_SECRET_ACCESS_KEY, TEST_ACCESS_KEY_SECRET);
@@ -137,6 +140,9 @@ public void testGetFreshCredentials() {
             Assert.assertEquals(TEST_ACCESS_KEY_SECRET, credentials.getSecretAccessKey());
             Assert.assertNull(credentials.getSecurityToken());
             Assert.assertFalse(credentials.useSecurityToken());
+            
+            new File(AuthUtils.DEFAULT_PROFILE_PATH).delete();
+            Thread.sleep(1000);
 
             // Fresh
             options = new HashMap<String, String>();
@@ -152,8 +158,6 @@ public void testGetFreshCredentials() {
             Assert.assertEquals(TEST_ACCESS_KEY_SECRET, credentials.getSecretAccessKey());
             Assert.assertEquals(TEST_SECURITY_TOKEN, credentials.getSecurityToken());
             Assert.assertTrue(credentials.useSecurityToken());
-
-            new File(AuthUtils.DEFAULT_PROFILE_PATH).delete();
         } catch (Exception e) {
             e.printStackTrace();
             Assert.fail(e.getMessage());

src/test/java/com/aliyun/oss/common/provider/STSAssumeRoleSessionCredentialsProviderTest.java

@@ -44,8 +44,8 @@ public void testStsAssumeRoleCredentialsProvider() {
             Assert.assertFalse(credentials.willSoonExpire());
             Assert.assertTrue(credentials.getAccessKeyId().startsWith("STS."));
             Assert.assertEquals(credentials.getAccessKeyId().length(), 29);
-            Assert.assertEquals(credentials.getSecretAccessKey().length(), 44);
-            Assert.assertEquals(credentials.getSecurityToken().length(), 544);
+            Assert.assertTrue(credentials.getSecretAccessKey().length() > 0);
+            Assert.assertTrue(credentials.getSecurityToken().length() > 0);
         } catch (Exception e) {
             e.printStackTrace();
             Assert.fail(e.getMessage());

src/test/java/com/aliyun/oss/common/provider/STSKeyPairSessionCredentialsProviderTest.java

@@ -98,7 +98,7 @@ public void testStsKeyPairCredentialsProviderNegative() throws ClientException,
 
             credentialsProvider = CredentialsProviderFactory.newSTSKeyPairSessionCredentialsProvider(TestConfig.RAM_REGION_ID,
                     publicKey.getPublicKeyId(), AuthUtils.loadPrivateKeyFromFile(TestConfig.PRIVATE_KEY_PATH))
-                    .withExpiredDuration(3601);
+                    .withExpiredDuration(100);
 
             Assert.assertNull(credentialsProvider.getCredentials());