Project

Profile

Help

Issue #8321

closed

check for the presence of trailing slash before assembling the url

Added by ipanova@redhat.com over 3 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 91
Quarter:

Description

Steps to reproduce in pulp2:

  1. create a file (iso) repo in pulp2, with an importer pointing to https://fixtures.pulpproject.org/file (NOTE the lack of trailing space)
  2. Sync the repository
  3. Migrate the repository to pulp3
  4. Check the created remote, you would expect: https://fixtures.pulpproject.org/file/PULP_MANIFEST, but in reality it will be https://fixtures.pulpproject.org/PULP_MANIFEST

place to fix https://github.com/pulp/pulp-2to3-migration/blob/master/pulp_2to3_migration/app/plugin/iso/repository.py#L37

Maybe also check for other occurrences where we use urljoin.

Also available in: Atom PDF