Project

Profile

Help

Backport #9526

Backport to 2.8: Pulp container does not validate container distribution base paths

Added by ipanova@redhat.com about 1 month ago. Updated 4 days ago.

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

100%

Estimated time:
Triaged:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Sprint 110
Quarter:

Description

Container distribution base paths are supposed to be a valid container repository names, however, the container distribution serializer doesn't validate for this aside from verifying the container namespace in the base path. Right now users can create distributions with names like i/am/an/very:bad:container/name.py, which breaks the registry in Automation Hub.


Related issues

Copied from Container Support - Issue #9403: Pulp container does not validate container distribution base pathsCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 360628aa View on GitHub
Added by Lubos Mjachky 4 days ago

Add validation for a repository base path

backports #9403 closes #9526 (cherry picked from commit 4d8802e6e928881dfc9282c990f4f6dea2d62048)

Revision 360628aa View on GitHub
Added by Lubos Mjachky 4 days ago

Add validation for a repository base path

backports #9403 closes #9526 (cherry picked from commit 4d8802e6e928881dfc9282c990f4f6dea2d62048)

History

#1 Updated by ipanova@redhat.com about 1 month ago

  • Copied from Issue #9403: Pulp container does not validate container distribution base paths added

#2 Updated by ipanova@redhat.com about 1 month ago

  • Sprint/Milestone set to 2.8.2

#3 Updated by rchan about 1 month ago

  • Sprint changed from Sprint 107 to Sprint 108

#4 Updated by ipanova@redhat.com about 1 month ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

#5 Updated by pulpbot about 1 month ago

  • Status changed from ASSIGNED to POST

#7 Updated by rchan 23 days ago

  • Sprint changed from Sprint 108 to Sprint 109

#8 Updated by rchan 8 days ago

  • Sprint changed from Sprint 109 to Sprint 110

#9 Updated by Anonymous 4 days ago

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

#10 Updated by pulpbot 4 days ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF