Project

Profile

Help

Issue #9403

closed

Pulp container does not validate container distribution base paths

Added by newswangerd about 3 years ago. Updated about 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 106
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 to Container Support - Backport #9526: Backport to 2.8: Pulp container does not validate container distribution base pathsCLOSED - CURRENTRELEASEipanova@redhat.com

Actions
Actions #2

Updated by ipanova@redhat.com about 3 years ago

  • Triaged changed from No to Yes
Actions #3

Updated by lmjachky about 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to lmjachky
  • Sprint set to Sprint 106
Actions #4

Updated by pulpbot about 3 years ago

  • Status changed from ASSIGNED to POST

Added by Lubos Mjachky about 3 years ago

Revision 4d8802e6 | View on GitHub

Add validation for a repository base path

closes #9403

Added by Lubos Mjachky about 3 years ago

Revision 4d8802e6 | View on GitHub

Add validation for a repository base path

closes #9403

Actions #5

Updated by Anonymous about 3 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by ipanova@redhat.com about 3 years ago

  • Sprint/Milestone set to 2.9.0
Actions #7

Updated by pulpbot about 3 years ago

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

Updated by ipanova@redhat.com about 3 years ago

  • Copied to Backport #9526: Backport to 2.8: Pulp container does not validate container distribution base paths added

Also available in: Atom PDF