Project

Profile

Help

Issue #481

Retrieving consumer history for a consumer that doesn't exist results in a 200 OK

Added by jcline@redhat.com over 4 years ago. Updated 7 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
API
Sprint/Milestone:
-
Start date:
Due date:
Severity:
1. Low
Version:
Master
Platform Release:
2.6.0
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Complexity:
Smash Test:
Verified:
Yes
Verification Required:
No
Sprint:

Description

Description of problem:
The docs (https://pulp-dev-guide.readthedocs.org/en/latest/integration/rest-api/consumer/history.html#retrieve-consumer-event-history) say a 404 should be returned if the consumer doesn't exist. However, a 200 and an empty list is actually returned.

Version-Release number of selected component (if applicable): master

How reproducible: always

Steps to Reproduce:
1. curl -i -H "Accept: application/json" -X GET -u admin:admin -k 'https://localhost/pulp/api/v2/consumers/NotARealConsumer/history/'

Actual results:
HTTP/1.1 200 OK
Date: Thu, 07 Aug 2014 15:25:56 GMT
Server: Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips mod_wsgi/3.5 Python/2.7.5
Content-Length: 2
Content-Type: application/json

[]

Expected results:
404 Not Found

Additional info:

+ This bug was cloned from Bugzilla Bug #1127817 +

History

#2 Updated by cduryee over 4 years ago

fixed in pulp 2.6.0-0.2.beta

+ This comment was cloned from Bugzilla #1127817 comment 2 +

#3 Updated by igulina@redhat.com over 4 years ago

rpm -qa pulp-server

pulp-server-2.6.0-0.2.beta.el6.noarch

curl -i -H "Accept: application/json" -X GET -u admin:admin -k 'https://localhost/pulp/api/v2/consumers/NotARealConsumer/history/'

HTTP/1.1 404 Not Found
Date: Tue, 13 Jan 2015 13:03:43 GMT
Server: Apache/2.2.15 (Red Hat)
Content-Length: 4
Connection: close
Content-Type: application/json

null

+ This comment was cloned from Bugzilla #1127817 comment 3 +

#4 Updated by bmbouter over 4 years ago

  • Severity changed from Low to 1. Low

#5 Updated by rbarlow over 4 years ago

  • Status changed from VERIFIED to CLOSED - CURRENTRELEASE

#6 Updated by pulpbot almost 3 years ago

  • Verified changed from No to Yes

#7 Updated by bmbouter 7 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF