Issue #3850
closed
Return relative URIs (e.g. /pulp/api/v3/repositories/1/) in the API
Status:
CLOSED - CURRENTRELEASE
Tags:
API Bindings, Katello
Description
The hrefs for objects returned from the bindings have full urls with hostname, port, etc. However, there are a couple reasons for this:
1. Katello has said their user base sometimes switches hostname/port/etc. So having full URIs means that they would be mutable.
2. The bindings require relative URIs
3. Full URIs are shorter and this means storing less data for third-party apps and such.
We should be returning relative URIs in our API for all hrefs. This includes places like created_resources in the task response.
- Description updated (diff)
- Triaged changed from No to Yes
- Sprint/Milestone set to 3.0.0
- Subject changed from Binding url formats are inconsistent to Binding URI formats are inconsistent
- Subject changed from Binding URI formats are inconsistent to Use relative URIs (e.g. /pulp/api/v3/repositories/file/1/)
- Description updated (diff)
- Subject changed from Use relative URIs (e.g. /pulp/api/v3/repositories/file/1/) to Return relative URIs (e.g. /pulp/api/v3/repositories/file/1/) in the API
- Description updated (diff)
- Groomed changed from No to Yes
- Sprint Candidate changed from No to Yes
This looks right to me. It's consistent with the recent postings of the id vs href special interest group. I'm marking as groomed and sprint candidate.
- Description updated (diff)
- Subject changed from Return relative URIs (e.g. /pulp/api/v3/repositories/file/1/) in the API to Return relative URIs (e.g. /pulp/api/v3/repositories/1/) in the API
- Status changed from NEW to ASSIGNED
- Assignee set to dkliban@redhat.com
- Status changed from ASSIGNED to POST
- Status changed from POST to MODIFIED
- Related to Test #4123: Test the generated API binds added
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
- Tags Katello added
- Tags deleted (
Katello-P1)
Also available in: Atom
PDF
Switches FileContent to using relative URLs for href
Required PR: https://github.com/pulp/pulp/pull/3607
re: #3850 https://pulp.plan.io/issues/3850