Merge pull request #21 from aliyun/optimizer

php sdk optimizer
aliyun · Mar 28, 2016 · 1a046a8 · 1a046a8
2 parents 4f6839d + 7470f3e
commit 1a046a8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/OSS/OssClient.php b/src/OSS/OssClient.php
@@ -65,6 +65,10 @@ class OssClient
      */
     public function __construct($accessKeyId, $accessKeySecret, $endpoint, $isCName = false, $securityToken = NULL)
     {
+    	$accessKeyId = trim($accessKeyId);
+	$accessKeySecret = trim($accessKeySecret);
+	$endpoint = trim(trim($endpoint), "/");
+
         if (empty($accessKeyId)) {
             throw new OssException("access key id is empty");
         }

diff --git a/tests/OSS/Tests/OssClientTest.php b/tests/OSS/Tests/OssClientTest.php
@@ -95,4 +95,17 @@ public function testConstrunct8()
         }
     }
 
+    public function testConstrunct9()
+    {
+        try {
+	    $accessKeyId = ' ' . getenv('OSS_ACCESS_KEY_ID') . ' ';
+	    $accessKeySecret = ' ' . getenv('OSS_ACCESS_KEY_SECRET') . ' ';
+	    $endpoint = ' ' . getenv('OSS_ENDPOINT') . '/ ';
+            $ossClient = new OssClient($accessKeyId, $accessKeySecret , $endpoint, false);
+            $ossClient->listBuckets();
+        } catch (OssException $e) {
+            $this->assertFalse(true);
+        }
+    }
+
 }