Merge pull request #259 from tencentyun/feature_jojoliang_0b77e284

预签名URL不编码

Author: agin719

cos.go

@@ -25,7 +25,7 @@ import (
 
 const (
 	// Version current go sdk version
-	Version               = "0.7.49"
+	Version               = "0.7.50"
 	UserAgent             = "cos-go-sdk-v5/" + Version
 	contentTypeXML        = "application/xml"
 	defaultServiceBaseURL = "http://service.cos.myqcloud.com"

object.go

@@ -130,7 +130,12 @@ func (s *ObjectService) GetPresignedURL(ctx context.Context, httpMethod, name, a
 	if name == "" {
 		return nil, fmt.Errorf("object key is empty.")
 	}
-	name = encodeURIComponent(name)
+	// 兼容 name 以 / 开头的情况
+	if strings.HasPrefix(name, "/") {
+		name = encodeURIComponent("/") + encodeURIComponent(name[1:], []byte{'/'})
+	} else {
+		name = encodeURIComponent(name, []byte{'/'})
+	}
 
 	sendOpt := sendOptions{
 		baseURL:   s.client.BaseURL.BucketURL,
@@ -196,8 +201,12 @@ func (s *ObjectService) GetPresignedURL2(ctx context.Context, httpMethod, name s
 	if name == "" {
 		return nil, fmt.Errorf("object key is empty.")
 	}
-
-	name = encodeURIComponent(name)
+	// 兼容 name 以 / 开头的情况
+	if strings.HasPrefix(name, "/") {
+		name = encodeURIComponent("/") + encodeURIComponent(name[1:], []byte{'/'})
+	} else {
+		name = encodeURIComponent(name, []byte{'/'})
+	}
 
 	cred := s.client.GetCredential()
 	if cred == nil {

object_test.go

@@ -258,6 +258,12 @@ func TestObjectService_GetPresignedURL(t *testing.T) {
 	if err == nil {
 		t.Errorf("GetPresignedURL expect err is not null")
 	}
+
+	_, err = client.Object.GetPresignedURL(c, http.MethodPut, "/", ak, sk, time.Hour, opt1)
+	if err != nil {
+		t.Errorf("GetPresignedURL return err: %v", err)
+	}
+
 }
 
 /*
@@ -371,6 +377,11 @@ func TestObjectService_GetPresignedURL2(t *testing.T) {
 		t.Errorf("GetPresignedURL expect err is not null")
 	}
 
+	_, err = client.Object.GetPresignedURL2(c, http.MethodPut, "/", time.Hour, opt1)
+	if err != nil {
+		t.Errorf("GetPresignedURL return err: %v", err)
+	}
+
 }
 
 func TestObjectService_GetPresignedURL3(t *testing.T) {