Project

Profile

Help

Actions
Send by e-mail Copy link

Story #6894

closed

As a user, I can use token auth in pulp_container 2.0.0

Added by dkliban@redhat.com over 4 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
mdellweg
Sprint/Milestone:
2.0.0
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 75
Quarter:

Description

The token authentication verification code needs to be refactored into a Django-REST-Framework compatible Authentication class.

Redirects to the content app need to contain a query parameter that will allow the content app to confirm that the user has permission to access the content it is requesting. This could possibly be the token also.

Once token auth is added back, all the tests that were disabled or modified need to be reverted to their original form so that token auth is being tested in CI. Revert this commit: https://github.com/pulp/pulp_container/commit/946eacfed12fa54aa89a3da293c4e03b3bb84580

Actions
Copy link
 #1

Updated by dkliban@redhat.com over 4 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by mdellweg over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg
Actions
Copy link
 #3

Updated by ipanova@redhat.com over 4 years ago

  • Sprint set to Sprint 74
Actions
Copy link
 #4

Updated by mdellweg over 4 years ago

I think the authenticating querystring to the content app should actually be verified by a content_guard.

Actions
Copy link
 #5

Updated by ipanova@redhat.com over 4 years ago

  • Project changed from Pulp to Container Support
Actions
Copy link
 #6

Updated by ipanova@redhat.com over 4 years ago

  • Sprint/Milestone set to 2.0.0
Actions
Copy link
 #7

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 74 to Sprint 75
Actions
Copy link
 #8

Updated by pulpbot over 4 years ago

  • Status changed from ASSIGNED to POST

Added by mdellweg over 4 years ago

Revision 02564d77 | View on GitHub

Add ContentRedirectContentGuard

fixes #6894 https://pulp.plan.io/issues/6894

Added by mdellweg over 4 years ago

Revision 02564d77 | View on GitHub

Add ContentRedirectContentGuard

fixes #6894 https://pulp.plan.io/issues/6894

Actions
Copy link
 #10

Updated by mdellweg over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions
Copy link
 #13

Updated by mdellweg over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF