S3 pre-signed URL: error with unicode filename

[PhucNguyen0311]

Hi, I got an issue relative to S3 storage with Unicode filename.

Currently, the CloudFront pre-signed method (generate_presigned_url_cloudfront) uses a function to get a nice filename but S3 isn't. So, I think it would be good if S3 pre-signed function (generate_presigned_url_s3) has the same method.

[marxjohnson]

@PhucNguyen0311 This looks like the right approach to me, however it introduces some duplicated logic. As you say, the cloudfront and s3 signing methods should do the same thing, so it would be good if they both did this by calling get_nice_filename(). Perhaps you could refactor that function slightly so that it returns the data in a format that can be used by both functions? This may require a small change to the cloudfront method too. For example, get_nice_filename() could return an array of content-disposition, filename and content-type, then generate_presigned_url_cloudfront() could use that to construct its query string while generate_presigned_url_s3() could use to it to construct its parameters.

[PhucNguyen0311]

@marxjohnson Thank you for your advice.
I updated the code for this fix. Please help me to review and let me know if you have any concerns.

[marxjohnson]

@PhucNguyen0311 This fix looks good to me, @brendanheywood what do you think?

[almadog]

Bumping this up - have been experiencing the same issue since we enabled s3 pre-signed urls.

[brendanheywood]

@PhucNguyen0311 can you please rebase your patch

[marxjohnson]

@brendanheywood I tried to contact @PhucNguyen0311 but didn't get a response, so I have done the rebase and created a new PR: #493