blob/azblob: Pass nil instead of an empty map for tags on Copy operat…

…ion (#2989)
google · Apr 19, 2021 · 448a545 · 448a545
1 parent 7b4db4c
commit 448a545
Show file tree

Hide file tree

Showing 5 changed files with 160 additions and 176 deletions.
diff --git a/blob/azureblob/azureblob.go b/blob/azureblob/azureblob.go
@@ -541,7 +541,6 @@ func (b *bucket) Copy(ctx context.Context, dstKey, srcKey string, opts *driver.C
 	mac := azblob.ModifiedAccessConditions{}
 	bac := azblob.BlobAccessConditions{}
 	at := azblob.AccessTierNone
-	btm := azblob.BlobTagsMap{}
 	if opts.BeforeCopy != nil {
 		asFunc := func(i interface{}) bool {
 			switch v := i.(type) {
@@ -561,7 +560,7 @@ func (b *bucket) Copy(ctx context.Context, dstKey, srcKey string, opts *driver.C
 			return err
 		}
 	}
-	resp, err := dstBlobURL.StartCopyFromURL(ctx, srcURL, md, mac, bac, at, btm)
+	resp, err := dstBlobURL.StartCopyFromURL(ctx, srcURL, md, mac, bac, at, nil /* BlobTagsMap */)
 	if err != nil {
 		return err
 	}

diff --git a/.../azureblob/testdata/TestConformance/TestAs/verify_As_returns_false_when_passed_nil.replay b/.../azureblob/testdata/TestConformance/TestAs/verify_As_returns_false_when_passed_nil.replay