Issue #4733: Pulp configured with S3 doesn't give users the Content-Disposition header - Pulp

Send by e-mail

Issue #4733

Pulp configured with S3 doesn't give users the Content-Disposition header

Added by bmbouter almost 2 years ago. Updated about 1 year ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

Category:

-

Sprint/Milestone:

Start date:

Due date:

Estimated time:

Severity:

2. Medium

Version:

Platform Release:

OS:

Triaged:

Yes

Groomed:

No

Sprint Candidate:

No

Tags:

Sprint:

Sprint 65

Quarter:

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
Added by bmbouter about 1 year ago

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.

closes #4733 https://pulp.plan.io/issues/4733

History

#1 Updated by bmbouter almost 2 years ago

Sprint/Milestone set to 3.0.0

#2 Updated by amacdona@redhat.com almost 2 years ago

Triaged changed from No to Yes

#3 Updated by daviddavis over 1 year ago

Sprint/Milestone deleted (~~3.0.0~~)

#4 Updated by newswangerd about 1 year 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 year ago

I've asked the question in django-storages upstream about how Pulp can configure this: https://github.com/jschneier/django-storages/issues/819

#6 Updated by bmbouter about 1 year ago

Status changed from NEW to ASSIGNED
Assignee set to bmbouter
Sprint set to Sprint 65

Adding to sprint because galaxy team is blocked by it.

#7 Updated by bmbouter about 1 year ago

Description updated (diff)

Adding solution description.

#8 Updated by bmbouter about 1 year ago

Status changed from ASSIGNED to POST

PR available at: https://github.com/pulp/pulpcore/pull/522

#9 Updated by bmbouter about 1 year ago

Status changed from POST to MODIFIED

Applied in changeset pulpcore|9f24cf976f683832313d6232651806a914dad443.

#10 Updated by bmbouter about 1 year ago

Sprint/Milestone set to 3.1.0

#11 Updated by bmbouter about 1 year ago

Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Send by e-mail

Also available in: Atom PDF