Skip to content

Update --allow-unordered to remove delimiter from request. #1350

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

visbits
Copy link

@visbits visbits commented Nov 9, 2023

Ceph does not support delimiter for allow-unordered: https://docs.ceph.com/en/quincy/radosgw/s3/bucketops/#id4

@visbits visbits changed the title Update S3.py Update --allow-unordered to remove delimiter from request. Nov 9, 2023
@fviard
Copy link
Contributor

fviard commented Jan 18, 2025

I'm not sure that the fix like that would be a good thing.
If you do that, it means that whatever will be your command, you will have all the subkeys when you use allow-unordered, like if you enabled recursive.
That could be unexpected and have side effect. I think, no?

Maybe a solution for this case, that would be a little bit more painful but safer, is to refuse commands that when you have the incompatible set "not recursive" and "allow-unordered" at the same time?

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.

2 participants