Issue #3063
closed
406 returned when a 404 should be
Description
Install pulpcore without any plugins.
Create repository.
http POST http://localhost:8000/api/v3/repositories/ name=foo2
Attempt to add file importer
(pulp) [vagrant@pulp3 pulp_example]$ http POST http://localhost:8000/api/v3/repositories/foo2/importers/file/ name='bar' download_policy='immediate' sync_mode='mirror' feed_url='https://repos.fedorapeople.org/pulp/pulp/demo_repos/test_file_repo/PULP_MANIFEST'
HTTP/1.0 406 Not Acceptable
Allow: GET, HEAD, OPTIONS
Content-Length: 57
Content-Type: application/coreapi+json
Date: Tue, 10 Oct 2017 17:29:50 GMT
Server: WSGIServer/0.2 CPython/3.5.4
Vary: Accept
X-Frame-Options: SAMEORIGIN
{
"detail": "Could not satisfy the request Accept header."
}
Downgrading djangorestframework to 3.6.4 and
Reverting urls.py to to 97093b071224700ff93fb6256292a53b424ddf1f commit produces the right response
git checkout 97093b071224700ff93fb6256292a53b424ddf1f platform/pulpcore/app/urls.py
(pulp) [vagrant@pulp3 pulp_example]$ http POST http://localhost:8000/api/v3/repositories/foo2/importers/file/ name='bar' download_policy='immediate' sync_mode='mirror' feed_url='https://repos.fedorapeople.org/pulp/pulp/demo_repos/test_file_repo/PULP_MANIFEST'
HTTP/1.0 404 Not Found
Content-Length: 17054
Content-Type: text/html
Date: Tue, 10 Oct 2017 17:35:53 GMT
Server: WSGIServer/0.2 CPython/3.5.4
X-Frame-Options: SAMEORIGIN
<!DOCTYPE html>
<html lang="en">
<head>
The follwoing commit broke things:
https://github.com/pulp/pulp/commit/8ecb2c2babe553014bde1a96232d0e6865068ef2
Updated by amacdona@redhat.com about 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to amacdona@redhat.com
- Priority changed from Normal to High
- Severity changed from 2. Medium to 3. High
- Triaged changed from No to Yes
- Tags Pulp 3 added
Updated by daviddavis about 7 years ago
- Status changed from ASSIGNED to POST
- Assignee changed from amacdona@redhat.com to daviddavis
Added by daviddavis about 7 years ago
Added by daviddavis about 7 years ago
Revision cfff931a | View on GitHub
Fix the api schema route
The issue was that we were mapping all /api/v3/ routes to the api schema which meant that all routes that didn't match another route displayed the api schema. It also caused some weird behaviors where we saw some 406s instead of 404. I also needed to move the route above the other routers because they assigned /api/v3/ to a route which returns a list of resources.
Updated by daviddavis about 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|cfff931ad86eba28d11bdc929287744e582e369a.
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Fix the api schema route
The issue was that we were mapping all /api/v3/ routes to the api schema which meant that all routes that didn't match another route displayed the api schema. It also caused some weird behaviors where we saw some 406s instead of 404. I also needed to move the route above the other routers because they assigned /api/v3/ to a route which returns a list of resources.
fixes #3063 https://pulp.plan.io/issues/3063