Project

Profile

Help

Issue #6154

closed

Redirect to S3 in pulp-content-app is broken in 3.1.0

Added by osapryki almost 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
4. Urgent
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 66
Quarter:

Description

pulp-content-app returns redirect to invalid S3 URL:

https://bucket.s3.amazonaws.com/artifact/xx/xxx?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=********&X-Amz-Date=20200213T141229Z&X-Amz-Expires=3600&X-Amz-SignedHeaders=host&X-Amz-Signature=********?response-content-disposition=attachment;+filename=pulp-artifact-filename

response-content-disposition is attached to the URL that already has query string which gives as a result invalid URL.

Please refer to for the correct example of setting Response-Content-Disposition parameter with django-storages:

https://github.com/ansible/galaxy/blob/devel/galaxy/api/download/views.py#L64

Actions #1

Updated by ipanova@redhat.com almost 5 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 66
Actions #2

Updated by fao89 almost 5 years ago

  • Status changed from NEW to ASSIGNED

Added by Fabricio Aguiar almost 5 years ago

Revision 3b917b46 | View on GitHub

Fixing broken S3 redirect

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

Actions #3

Updated by fao89 almost 5 years ago

  • Status changed from ASSIGNED to POST
Actions #4

Updated by Anonymous almost 5 years ago

  • Status changed from POST to MODIFIED

Added by Fabricio Aguiar almost 5 years ago

Revision 68a9b12c | View on GitHub

Fixing broken S3 redirect

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

(cherry picked from commit 3b917b4639651913bfb0df3a80869c2996867296)

Actions #5

Updated by Anonymous almost 5 years ago

Actions #6

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #7

Updated by bmbouter almost 5 years ago

  • Sprint/Milestone set to 3.1.1

Also available in: Atom PDF