Project

Profile

Help

Issue #6227

closed

Pulp sends auth credentials to redirect

Added by daviddavis 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:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

S3 is returning a 400 error when trying to sync collections from Automation Hub. The problem is that the basic auth credentials set on the remote are getting sent to S3. Pulp is making a call to Automation Hub with the necessary basic auth credentials and then AH is redirecting Pulp to S3. Pulp is sending these basic auth credentials to S3 when it should not. Looks like if we set the credentials on the request instead of the session, aiohttp will only send the credentials to redirected locations when the domains match which is the behavior we want.

Actions #1

Updated by daviddavis almost 5 years ago

  • Description updated (diff)

Added by daviddavis almost 5 years ago

Revision 4505f145 | View on GitHub

Stop sending basic auth credentials to redirect location

Pulp is getting redirected and sending the Remote's credentials to the redirected location. Looks like if we set the credentials on the request instead of the session, aiohttp will only send the credentials to redirected locations when the domains match which is the behavior we want.

fixes #6227

Actions #2

Updated by daviddavis almost 5 years ago

  • Status changed from ASSIGNED to MODIFIED
Actions #3

Updated by daviddavis almost 5 years ago

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

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.2.0

Also available in: Atom PDF