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

test_bucket_list_prefix_basic and test_bucket_list_prefix_delimiter_basic needed bypass #481

Open
shirady opened this issue Dec 18, 2022 · 1 comment

Comments

@shirady
Copy link

shirady commented Dec 18, 2022

Hi,
I found out that the prefix response is not decoded back in the field of prefix when using list_objects.
As you probably know when using boto3 the client sets automatically the encoding type url (see boto/boto3#816):

“Encoding type is automatically set because there are unicode characters which can break the XML parser. To work around the issue, we automatically set the encoding type parameter to url if the customer hasn't already set it. We then decode the keys on the way back, again only if we provided that parameter.”

That why is needs to decode the fields back. I found out that the prefix response is not decoded back in the field of a prefix when using list_objects, I opened boto3 an issue about it, but until they fix it we can use a bypass for the problematic line:
eq(response['Prefix'], 'foo/')

I suggest that we will decode response['Prefix'] and insert a comment so that it will be clear that it is just a bypass.
This case when using the list_objects with prefix, appears in 2 tests: test_bucket_list_prefix_basic and test_bucket_list_prefix_delimiter_basic

@shirady
Copy link
Author

shirady commented Jan 12, 2023

An update from boto3 issue - this might be due to boto3 version which is 1.23.10 (whereas in 1.23.33 for the latest version they did not reproduce this bug).

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

No branches or pull requests

1 participant