-
Notifications
You must be signed in to change notification settings - Fork 631
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
Fix listObjects hang when listing special characters #1165
Conversation
To reproduce
This will hang forever listing first 1000 entries repeatedly. |
9948cdf
to
0ffee18
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@harshavardhana NextContinuationToken needs also to be unescaped in ListObjectsV2
@vadmeste looks like that is only for MinIO since it's an opaque value in AWS S3 |
This PR 1e01719 introduced a regression in handling special character file listing, this causes `mc ls` like commands to hang. Fixes minio/mc#2903
0ffee18
to
78172f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM & tested
This PR 1e01719 introduced
a regression in handling special character file listing, this
causes
mc ls
like commands to hang.Fixes minio/mc#2903