Browse Source

fix roa request

yixiong.jxy 7 years ago
parent
commit
b42e459846
2 changed files with 6 additions and 4 deletions
  1. 3 3
      sdk/auth/roa_signature_composer.go
  2. 3 1
      sdk/requests/roa_request.go

+ 3 - 3
sdk/auth/roa_signature_composer.go

@@ -42,9 +42,9 @@ func completeROASignParams(request requests.AcsRequest, signer Signer, regionId
 
 
 	// complete query params
 	// complete query params
 	queryParams := request.GetQueryParams()
 	queryParams := request.GetQueryParams()
-	if _, ok := queryParams["RegionId"]; !ok {
-		queryParams["RegionId"] = regionId
-	}
+	//if _, ok := queryParams["RegionId"]; !ok {
+	//	queryParams["RegionId"] = regionId
+	//}
 	if extraParam := signer.GetExtraParam(); extraParam != nil {
 	if extraParam := signer.GetExtraParam(); extraParam != nil {
 		for key, value := range extraParam {
 		for key, value := range extraParam {
 			if key == "SecurityToken" {
 			if key == "SecurityToken" {

+ 3 - 1
sdk/requests/roa_request.go

@@ -77,7 +77,9 @@ func (request *RoaRequest) buildQueries(needParamEncode bool) string {
 	// append urlBuilder
 	// append urlBuilder
 	urlBuilder := bytes.Buffer{}
 	urlBuilder := bytes.Buffer{}
 	urlBuilder.WriteString(path)
 	urlBuilder.WriteString(path)
-	urlBuilder.WriteString("?")
+	if len(queryKeys) > 0 {
+		urlBuilder.WriteString("?")
+	}
 	for i := 0; i < len(queryKeys); i++ {
 	for i := 0; i < len(queryKeys); i++ {
 		queryKey := queryKeys[i]
 		queryKey := queryKeys[i]
 		urlBuilder.WriteString(queryKey)
 		urlBuilder.WriteString(queryKey)