Bug 766012

Summary:	Attempting to delete non-existent configs throws a 500 instead of 404
Product:	[Retired] CloudForms Cloud Engine	Reporter:	Greg Blomquist <gblomqui>
Component:	aeolus-configserver	Assignee:	Greg Blomquist <gblomqui>
Status:	CLOSED CURRENTRELEASE	QA Contact:	dgao
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	akarol, deltacloud-maint
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-08-30 17:17:22 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:	783201
Bug Blocks:

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