Issue #4733
Pulp configured with S3 doesn't give users the Content-Disposition header
Description
Problem¶
After Pulp redirects the user to Amazon S3 to download the file the user receives it but it does not have the correct filename. The Artifact filename is something stored in Pulp's database. It's also possible that one Artifact is present in multiple repos as various filenames so this needs to be set as a per-request piece of data.
Solution¶
At the redirect time, we'll include the response-content-disposition=attachment; filename=abc.doc'; fragment. This is similar to what Galaxy already did before: https://github.com/ansible/galaxy/blob/devel/galaxy/api/download/views.py#L65
Associated revisions
Revision 9f24cf97
View on GitHub
Have S3 and Azure redirect specify filename
The filename is specified as part of the redirect url.
This also changes the Handler's interface for the plugin writer, and there is a changelog entry with more details.
History
#2
Updated by amacdona@redhat.com 10 months ago
- Triaged changed from No to Yes
#3
Updated by daviddavis 6 months ago
- Sprint/Milestone deleted (
3.0.0)
#4
Updated by newswangerd about 1 month ago
We're adding a download button in Automation Hub and this issue is causing all of our downloads in the browser to have unintelligible filenames.
#5
Updated by bmbouter about 1 month ago
I've asked the question in django-storages upstream about how Pulp can configure this: https://github.com/jschneier/django-storages/issues/819
#8
Updated by bmbouter 30 days ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/pulpcore/pull/522
#9
Updated by bmbouter 29 days ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|9f24cf976f683832313d6232651806a914dad443.
Please register to edit this issue
.