chore: add support for copyloopvar linter (#2709)

Author: remyleone

.golangci.yml

@@ -9,6 +9,7 @@ linters:
     - bodyclose # checks whether HTTP response body is closed successfully [fast: false, auto-fix: false]
     - containedctx # containedctx is a linter that detects struct contained context.Context field [fast: true, auto-fix: false]
     - contextcheck # check the function whether use a non-inherited context [fast: false, auto-fix: false]
+    - copyloopvar # copyloopvar is a linter detects places where loop variables are copied [fast: true, auto-fix: false]
     - decorder # check declaration order and count of types, constants, variables and functions [fast: true, auto-fix: false]
     - dogsled # Checks assignments with too many blank identifiers (e.g. x, _, _, _, := f()) [fast: true, auto-fix: false]
     - dupword # checks for duplicate words in the source code [fast: true, auto-fix: true]

internal/services/object/helpers_object.go

@@ -372,8 +372,6 @@ func deleteS3ObjectVersions(ctx context.Context, conn *s3.S3, bucketName string,
 		pool := workerpool.NewWorkerPool(deletionWorkers)
 
 		for _, objectVersion := range page.Versions {
-			objectVersion := objectVersion
-
 			pool.AddTask(func() error {
 				objectKey := aws.StringValue(objectVersion.Key)
 				objectVersionID := aws.StringValue(objectVersion.VersionId)
@@ -417,8 +415,6 @@ func deleteS3ObjectVersions(ctx context.Context, conn *s3.S3, bucketName string,
 		pool := workerpool.NewWorkerPool(deletionWorkers)
 
 		for _, deleteMarkerEntry := range page.DeleteMarkers {
-			deleteMarkerEntry := deleteMarkerEntry
-
 			pool.AddTask(func() error {
 				deleteMarkerKey := aws.StringValue(deleteMarkerEntry.Key)
 				deleteMarkerVersionsID := aws.StringValue(deleteMarkerEntry.VersionId)