Project

Profile

Help

Story #6894

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

Added by dkliban@redhat.com over 1 year ago. Updated about 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
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

History

#1 Updated by dkliban@redhat.com over 1 year ago

  • Description updated (diff)

#2 Updated by mdellweg over 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg

#3 Updated by ipanova@redhat.com over 1 year ago

  • Sprint set to Sprint 74

#4 Updated by mdellweg over 1 year ago

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

#5 Updated by ipanova@redhat.com over 1 year ago

  • Project changed from Pulp to Container Support

#6 Updated by ipanova@redhat.com over 1 year ago

  • Sprint/Milestone set to 2.0.0

#7 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 74 to Sprint 75

#8 Updated by pulpbot over 1 year ago

  • Status changed from ASSIGNED to POST

#10 Updated by mdellweg over 1 year ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#13 Updated by mdellweg about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF