openstack/swift
Code Issues Proposed changes

Proof that test "coverage" != no obvious bugs

Browse Source
This commit is contained in:
gholt 2010-09-10 07:52:10 -07:00
parent 6b353a520f
commit 7057a7c3b3
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
swift/proxy/server.py
View File

@ -851,6 +851,7 @@ class ContainerController(Controller):
req.headers[header])
except ValueError, err:
return HTTPBadRequest(request=req, body=str(err))
return None
def GETorHEAD(self, req):
"""Handler for HTTP GET/HEAD requests."""
@ -882,8 +883,8 @@ class ContainerController(Controller):
@public
def PUT(self, req):
"""HTTP PUT request handler."""
self.clean_acls(req)
error_response = check_metadata(req, 'container')
error_response = \
self.clean_acls(req) or check_metadata(req, 'container')
if error_response:
return error_response
if len(self.container_name) > MAX_CONTAINER_NAME_LENGTH:
@ -948,8 +949,8 @@ class ContainerController(Controller):
@public
def POST(self, req):
"""HTTP POST request handler."""
self.clean_acls(req)
error_response = check_metadata(req, 'container')
error_response = \
self.clean_acls(req) or check_metadata(req, 'container')
if error_response:
return error_response
account_partition, accounts = self.account_info(self.account_name)