Don't request metadata when listing objects

While some implementations may support it, it appears that most don't, and some may in fact return an error if it is requested. We already stat the object to get the metadata anyway, so this was unnecessary if harmless on most implementations. Signed-off-by: Brad Davidson <brad.davidson@rancher.com>

Don't request metadata when listing objects
a92c4a0f · Brad Davidson · Brad Davidson · 96ebb963 · a92c4a0f
Commit a92c4a0f authored Nov 21, 2023 by Brad Davidson Committed by Brad Davidson Nov 21, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

s3.go pkg/etcd/s3.go +2 -3

No files found.
--- a/pkg/etcd/s3.go
+++ b/pkg/etcd/s3.go
@@ -360,9 +360,8 @@ func (s *S3) listSnapshots(ctx context.Context) (map[string]snapshotFile, error)
 	defer cancel()

 	opts := minio.ListObjectsOptions{
-		Prefix:       s.config.EtcdS3Folder,
-		Recursive:    true,
-		WithMetadata: true,
+		Prefix:    s.config.EtcdS3Folder,
+		Recursive: true,
 	}

 	objects := s.client.ListObjects(ctx, s.config.EtcdS3BucketName, opts)