Project

Profile

Help

Issue #2985

I can create importers/publishers for any repo while targeting a specific repo URL

Added by bizhang almost 3 years ago. Updated 5 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 25

Description

I have two repositories- repo1 and repo2. If I POST to the repo1 URL to create an importer/publisher, I can target repo2 with the repository field so that the importer/publisher gets created on repo2 instead.

http --auth admin:admin --json POST http://127.0.0.1:3000/api/v3/repositories/repo1/publishers/file/ name='publisher1' feed_url='asdfasdf' download_policy='IMMEDIATE' repository='http://127.0.0.1:3000:3000/api/v3/repositories/repo2/'

This is not what I would expect from a RESTful API; I would expect any request to the repo1 url to target the repo1 resource, and not have to pass an additional repository parameter.

Associated revisions

Revision 5c7aed9a View on GitHub
Added by werwty over 2 years ago

Set the href_writable on RepositoryRelatedFields

The parent repo will now be determined by the url instead of by request parameters.

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

Revision 5c7aed9a View on GitHub
Added by werwty over 2 years ago

Set the href_writable on RepositoryRelatedFields

The parent repo will now be determined by the url instead of by request parameters.

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

History

#1 Updated by ttereshc almost 3 years ago

  • Sprint/Milestone set to 43
  • Triaged changed from No to Yes

This issue is likely present for any nested URLs.

#2 Updated by bizhang over 2 years ago

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

#3 Updated by jortel@redhat.com over 2 years ago

  • Sprint/Milestone changed from 43 to 44

#4 Updated by bizhang over 2 years ago

  • Status changed from ASSIGNED to POST

#5 Updated by werwty over 2 years ago

  • Status changed from POST to MODIFIED

#6 Updated by bmbouter about 2 years ago

  • Sprint set to Sprint 25

#7 Updated by bmbouter about 2 years ago

  • Sprint/Milestone deleted (44)

#8 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#9 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#10 Updated by bmbouter 5 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF