Project

Profile

Help

Issue #3632

Pulp_python does not normalize project name before publish

Added by bizhang over 3 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version - Python:
Platform Release:
2.16.4
Target Release - Python:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 38
Quarter:

Description

pip install -i pulp.repo trusted-host pulp.repo CouchDB
Is trying to download "/pulp/python/web/ghostdev/simple/couchdb/" but pulp mirrored it to http://pulp.repo/pulp/python/web/ghostdev/simple/CouchDB/

From PEP 503 it seems that it must be lowercases : https://www.python.org/dev/peps/pep-0503/

We should normalize the project name prior to publication

Associated revisions

Revision 7096aab0 View on GitHub
Added by werwty over 3 years ago

normalize project name before publish

closes #3632 https://pulp.plan.io/issues/3632

Revision ffb575c4 View on GitHub
Added by werwty over 3 years ago

normalize project name before publish

closes #3632 https://pulp.plan.io/issues/3632

(cherry picked from commit 7096aab0bb68f7dc3c025198e25af91fda3439b3)

History

#1 Updated by bizhang over 3 years ago

  • Subject changed from Pulp does not normalize project name before publish to Pulp_python does not normalize project name before publish

#2 Updated by dalley over 3 years ago

We can re-use the function that Pulp 3 python uses to do so

#3 Updated by amacdona@redhat.com over 3 years ago

Should we normalize before publication, or should we normalize when we store it? I think we probably want to normalize project names on the remote and project name on the PythonPackageContent model.

#4 Updated by dalley over 3 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 36

#5 Updated by rchan over 3 years ago

  • Sprint changed from Sprint 36 to Sprint 37

#6 Updated by rchan over 3 years ago

  • Sprint changed from Sprint 37 to Sprint 38

#7 Updated by bizhang over 3 years ago

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

#8 Updated by bizhang over 3 years ago

  • Status changed from ASSIGNED to POST

#9 Updated by werwty over 3 years ago

  • Status changed from POST to MODIFIED

#10 Updated by daviddavis over 3 years ago

  • Platform Release set to 2.16.4

#11 Updated by werwty over 3 years ago

#12 Updated by daviddavis over 3 years ago

  • Status changed from MODIFIED to 5

#13 Updated by daviddavis over 3 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#14 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF