Skip to content

Commit

Permalink
fix common prefixes returned by List method of s3 client (grafana#3358)
Browse files Browse the repository at this point in the history
* fix common prefixes returned by List method of s3

Signed-off-by: Sandeep Sukhani <sandeep.d.sukhani@gmail.com>

* update changelog

Signed-off-by: Sandeep Sukhani <sandeep.d.sukhani@gmail.com>

* changes suggested from PR review

Signed-off-by: Sandeep Sukhani <sandeep.d.sukhani@gmail.com>
  • Loading branch information
sandeepsukhani authored Oct 21, 2020
1 parent aed2469 commit 3f8a530
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion aws/s3_storage_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@ func (a *S3ObjectClient) List(ctx context.Context, prefix, delimiter string) ([]
}

for _, commonPrefix := range output.CommonPrefixes {
commonPrefixes = append(commonPrefixes, chunk.StorageCommonPrefix(commonPrefix.String()))
commonPrefixes = append(commonPrefixes, chunk.StorageCommonPrefix(aws.StringValue(commonPrefix.Prefix)))
}

if !*output.IsTruncated {
Expand Down

0 comments on commit 3f8a530

Please sign in to comment.