Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve ListObject Compatibility #18099

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

klauspost
Copy link
Contributor

@klauspost klauspost commented Sep 25, 2023

Description

Do not error out when a provided marker is before or after the prefix, but instead just ignore it if before and return an empty list when after.

Fixes #18093

How to test this PR?

In-repo tests adjusted.

mint may have to be adjusted for this, waiting for results Mint also ok.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Unit tests added/updated

Do not error out when a provided marker is before or after the prefix, but instead just ignore it if before and return an empty list when after.

Fixes minio#18093

(mint may have to be adjusted for this, waiting for results)
@harshavardhana harshavardhana merged commit 21e8e07 into minio:master Sep 25, 2023
19 checks passed
@klauspost klauspost deleted the relax-listing-constraints branch September 25, 2023 15:25
Jamstah added a commit to Jamstah/distribution that referenced this pull request Oct 17, 2023
This fixes some of the tests for minio.

The walk tests needs a version of minio that contains minio/minio#18099

The storage classes minio supports are a subset of the s3 classes.

Signed-off-by: James Hewitt <james.hewitt@uk.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ListObjectsV2: Invalid combination of marker and prefix when no Marker is set
2 participants