Issue #5486
closedPlugin url patterns allow an arbitrary base path (not just /pulp/api/v3/)
Description
Original:
In each of their generated docs I noticed the endpoints all looked like
**POST** /docker/blah
instead of
**POST** /pulp/api/v3/blah
. Also, when doing DockerRecursiveAddApi.create I get a 404 and
No route matches [POST] "/docker/recursive-add"
at
lib/pulp_docker_client/api_client.rb:81:in `call_api'
However, it looks like the schema is actually correct, but the urlmatching is not strict enough.
All of these are valid and routed to the same view:
http POST localhost:24817/docker/recursive-add/ repository=/pulp/api/v3/repositories/597c2209-2ff2-4eec-b476-3740ff2879de/
http POST localhost:24817/pulp/api/v3/docker/recursive-add/ repository=/pulp/api/v3/repositories/597c2209-2ff2-4eec-b476-3740ff2879de/
http POST localhost:24817/thanks/for/reading/the/details/docker/recursive-add/ repository=/pulp/api/v3/repositories/597c2209-2ff2-4eec-b476-3740ff2879de/
IIRC, this is actually a deliberate feature of the plugin api, so they can register non-prefixed endpoints like /v2/ for Docker.
This might just be as simple as adding "^pulp/api/v3/" to the start of the urlpatterns, but it will need to be done for any plugins that have implemented custom urls.
Related issues
Updated by amacdona@redhat.com about 5 years ago
- Project changed from Docker Support to Pulp
- Subject changed from DockerRecursiveAdd, DockerRecursiveRemove, and DockerCopy ruby bindings go to the wrong url to Plugin url patterns allow an arbitrary base path (not just /pulp/api/v3/)
- Description updated (diff)
Updated by amacdona@redhat.com about 5 years ago
This seems fixes the issue (forcing the use of /pulp/api/v3/) for pulp_docker. https://github.com/pulp/pulp_docker/pull/415/
Updated by amacdona@redhat.com about 5 years ago
- Description updated (diff)
- Tags deleted (
API Bindings)
Updated by ipanova@redhat.com about 5 years ago
- Related to Issue #5330: Hardcoded URL patterns needs proper check added
Updated by ipanova@redhat.com about 5 years ago
- Status changed from NEW to POST
- Assignee set to amacdona@redhat.com
- Sprint set to Sprint 59
Updated by ipanova@redhat.com about 5 years ago
- Project changed from Pulp to Docker Support
I am moving this back to docker since we fixed this only in docker plugin, other plugins would need to submit separate PRs and/or issues
Added by amacdona@redhat.com about 5 years ago
Added by amacdona@redhat.com about 5 years ago
Revision f063981f | View on GitHub
Add url prefix to plugin urls
Added by amacdona@redhat.com about 5 years ago
Revision f063981f | View on GitHub
Add url prefix to plugin urls
Added by amacdona@redhat.com about 5 years ago
Revision f063981f | View on GitHub
Add url prefix to plugin urls
Updated by ipanova@redhat.com about 5 years ago
- Status changed from POST to MODIFIED
Updated by ipanova@redhat.com about 5 years ago
- Project changed from Docker Support to Container Support
Updated by ipanova@redhat.com almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Add url prefix to plugin urls
https://pulp.plan.io/issues/5486 re #5486