Bug 766012 - Attempting to delete non-existent configs throws a 500 instead of 404
Summary: Attempting to delete non-existent configs throws a 500 instead of 404
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-configserver
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Greg Blomquist
QA Contact: dgao
URL:
Whiteboard:
Depends On: 783201
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-09 21:14 UTC by Greg Blomquist
Modified: 2012-08-30 17:17 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:17:22 UTC

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Greg Blomquist 2011-12-09 21:14:03 UTC 
Description of problem:

Issuing an HTTP DELETE to the config server for /deployable/$ver/$uuid, where $uuid does not exist throws a 500 error.

The actual response should be a 404 error.

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

aeolus-configserver-0.4.2
Comment 1 Greg Blomquist 2011-12-09 21:26:54 UTC 
Patch posted and pushed.

New config server version:

aeolus-configserver-0.4.3

https://fedorahosted.org/pipermail/aeolus-devel/2011-December/007279.html
Comment 2 Greg Blomquist 2012-01-11 16:05:57 UTC 
https://github.com/aeolusproject/audrey/commit/5c9e789c88e4c0bd68f54533ce66decf4c2b1fb9
Comment 3 dgao 2012-01-24 20:19:04 UTC 
thin.log
10.11.230.249 - - [24/Jan/2012 15:17:45] "DELETE /deployable/1/test_uuid_1/non_functional_uuid HTTP/1.1" 404 18 0.0012

test client side:

DELETE /deployable/1/test_uuid_1/non_functional_uuid HTTP/1.1
Receiving response: HTTP/1.1 404 Not Found
Note You need to log in before you can comment on or make changes to this bug.