Project

Profile

Help

Issue #6507

closed

The base_url of a Distribution is missing a trailing slash

Added by dkliban@redhat.com over 4 years ago. Updated over 4 years ago.

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

Description

Since the content app requires a trailing slash at the end of a URL, the base_url for a Distribution should include it. Without it the user receives a 404.

The API currently returns something like this:

        {
            "base_path": "pub/centos/8/BaseOS/x86_64/kickstart",
            "base_url": "http://localhost:24816/pulp/content/pub/centos/8/BaseOS/x86_64/kickstart",
            "content_guard": null,
            "name": "5e97b2d16e50c92d5c2d60bc-centos8",
            "publication": "/pulp/api/v3/publications/rpm/rpm/2c62294c-6e2d-4460-9358-0efc4f3e01a7/",
            "pulp_created": "2020-04-16T14:11:09.474847Z",
            "pulp_href": "/pulp/api/v3/distributions/rpm/rpm/a2dce49e-925f-4965-be4c-a2c6df62350b/"
        }

But it should return

        {
            "base_path": "pub/centos/8/BaseOS/x86_64/kickstart",
            "base_url": "http://localhost:24816/pulp/content/pub/centos/8/BaseOS/x86_64/kickstart/",
            "content_guard": null,
            "name": "5e97b2d16e50c92d5c2d60bc-centos8",
            "publication": "/pulp/api/v3/publications/rpm/rpm/2c62294c-6e2d-4460-9358-0efc4f3e01a7/",
            "pulp_created": "2020-04-16T14:11:09.474847Z",
            "pulp_href": "/pulp/api/v3/distributions/rpm/rpm/a2dce49e-925f-4965-be4c-a2c6df62350b/"
        }
Actions #1

Updated by fao89 over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 70
Actions #2

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 70 to Sprint 71
Actions #3

Updated by lmjachky over 4 years ago

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

Added by Lubos Mjachky over 4 years ago

Revision 9df02abd | View on GitHub

Add a missing trailing slash

closes #6507

Actions #4

Updated by pulpbot over 4 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by Anonymous over 4 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by dkliban@redhat.com over 4 years ago

  • Sprint/Milestone set to 3.4.0
Actions #7

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF