Fixing S3 Copy Object Go SDK v1/v2 example to encode url

awsdocs#740

Author: haixinchen

go/example_code/s3/s3_copy_object.go

@@ -32,7 +32,8 @@ import (
     "github.com/aws/aws-sdk-go/aws/session"
     "github.com/aws/aws-sdk-go/service/s3"
     "fmt"
-    "os"
+	"net/url"
+	"os"
 )
 
 // Copies the item in the bucket to another bucket.
@@ -60,7 +61,8 @@ func main() {
     svc := s3.New(sess)
 
     // Copy the item
-    _, err = svc.CopyObject(&s3.CopyObjectInput{Bucket: aws.String(other), CopySource: aws.String(source), Key: aws.String(item)})
+	_, err = svc.CopyObject(&s3.CopyObjectInput{Bucket: aws.String(other),
+		CopySource: aws.String(url.PathEscape(source)), Key: aws.String(item)})
     if err != nil {
         exitErrorf("Unable to copy item from bucket %q to bucket %q, %v", bucket, other, err)
     }

go/s3/CopyObject/CopyObject.go

@@ -18,8 +18,9 @@ package main
 import (
     "flag"
     "fmt"
+	"net/url"
 
-    "github.com/aws/aws-sdk-go/aws"
+	"github.com/aws/aws-sdk-go/aws"
     "github.com/aws/aws-sdk-go/aws/session"
     "github.com/aws/aws-sdk-go/service/s3"
 )
@@ -42,7 +43,7 @@ func CopyItem(sess *session.Session, sourceBucket *string, targetBucket *string,
     // Copy the item
     _, err := svc.CopyObject(&s3.CopyObjectInput{
         Bucket:     targetBucket,
-        CopySource: aws.String(source),
+		CopySource: aws.String(url.PathEscape(source)),
         Key:        item,
     })
     // snippet-end:[s3.go.copy_object.call]

gov2/s3/CopyObject/CopyObjectv2.go

@@ -7,9 +7,11 @@ import (
 	"context"
 	"flag"
 	"fmt"
+	"net/url"
 
 	"github.com/aws/aws-sdk-go-v2/config"
 	"github.com/aws/aws-sdk-go-v2/service/s3"
+	"github.com/aws/aws-sdk-go/aws"
 )
 
 // S3CopyObjectAPI defines the interface for the Amazon Simple Storage Service (Amazon S3) CopyObject function.
@@ -53,7 +55,7 @@ func main() {
 	client := s3.NewFromConfig(cfg)
 
 	input := &s3.CopyObjectInput{
-		Bucket:     sourceBucket,
+		Bucket:     aws.String(url.PathEscape(*sourceBucket)),
 		CopySource: destinationBucket,
 		Key:        objectName,
 	}

gov2/s3/CopyObject/CopyObjectv2_test.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"errors"
 	"io/ioutil"
+	"net/url"
 	"testing"
 	"time"
 
@@ -78,7 +79,7 @@ func TestCopyObject(t *testing.T) {
 
 	// Build the request with its input parameters
 	input := s3.CopyObjectInput{
-		CopySource: &globalConfig.SourceBucket,
+		CopySource: aws.String(url.PathEscape(globalConfig.SourceBucket)),
 		Bucket:     &globalConfig.DestinationBucket,
 		Key:        &globalConfig.ObjectKey,
 	}