fix getCnameExcludeList bug in the multithread.

Author: huiguangjun

src/main/java/com/aliyun/oss/ClientConfiguration.java

@@ -99,6 +99,11 @@ public class ClientConfiguration {
 
     protected SignVersion signatureVersion = DEFAULT_SIGNATURE_VERSION;
 
+    public ClientConfiguration() {
+        super();
+        AppendDefaultExcludeList(this.cnameExcludeList);
+    }
+
     /**
      * Gets the user agent string.
      *
@@ -420,13 +425,6 @@ public void setProtocol(Protocol protocol) {
      * @return The excluded CName list, immutable.
      */
     public List<String> getCnameExcludeList() {
-        if (this.cnameExcludeList.size() == 0) {
-            rlock.lock();
-            if (this.cnameExcludeList.size() == 0) {
-                AppendDefaultExcludeList(this.cnameExcludeList);
-            }
-            rlock.unlock();
-        }
         return Collections.unmodifiableList(this.cnameExcludeList);
     }
 

src/test/java/com/aliyun/oss/common/comm/OSSClientTest.java

@@ -166,7 +166,11 @@ public void testClientConfiguration() {
         conf.setSignerHandlers(null);
 
         List<String> cnameList = conf.getCnameExcludeList();
-        conf.setCnameExcludeList(cnameList);
+        assertEquals(3, cnameList.size());
+        assertEquals(true, cnameList.contains("aliyuncs.com"));
+        assertEquals(true, cnameList.contains("aliyun-inc.com"));
+        assertEquals(true, cnameList.contains("aliyun.com"));
+
         cnameList = new ArrayList<String>();
         cnameList.add("");
         cnameList.add("cname");
@@ -177,6 +181,15 @@ public void testClientConfiguration() {
         assertEquals(5, gCnameList.size());
         assertEquals(true, gCnameList.contains("cname"));
         assertEquals(true, gCnameList.contains("cname1"));
+        assertEquals(true, gCnameList.contains("aliyun-inc.com"));
+
+        cnameList = new ArrayList<String>();
+        conf.setCnameExcludeList(cnameList);
+        gCnameList = conf.getCnameExcludeList();
+        assertEquals(3, gCnameList.size());
+        assertEquals(true, gCnameList.contains("aliyuncs.com"));
+        assertEquals(true, gCnameList.contains("aliyun-inc.com"));
+        assertEquals(true, gCnameList.contains("aliyun.com"));
 
         try {
             conf.setCnameExcludeList(null);

src/test/java/com/aliyun/oss/integrationtests/RtmpTest.java

@@ -622,7 +622,7 @@ public void testGeneratePushflowUri() {
 
             OSS client = new OSSClientBuilder().build("https://endpoint/", "ak", "sk", "sts");
             uri = client.generateRtmpUri("bucket", "live", "play.m3u8", 1000 );
-            Assert.assertTrue(uri.startsWith("rtmp://" + bucketName));
+            Assert.assertTrue(uri.startsWith("rtmp://bucket"));
 
         } catch (Exception e) {
             Assert.fail(e.getMessage());