Merge "Handle deleted files with if-none-match"
This commit is contained in:
@@ -686,7 +686,7 @@ class ObjectController(BaseStorageServer):
|
|||||||
except DiskFileXattrNotSupported:
|
except DiskFileXattrNotSupported:
|
||||||
return HTTPInsufficientStorage(drive=device, request=request)
|
return HTTPInsufficientStorage(drive=device, request=request)
|
||||||
except DiskFileDeleted as e:
|
except DiskFileDeleted as e:
|
||||||
orig_metadata = e.metadata
|
orig_metadata = {}
|
||||||
orig_timestamp = e.timestamp
|
orig_timestamp = e.timestamp
|
||||||
except (DiskFileNotExist, DiskFileQuarantined):
|
except (DiskFileNotExist, DiskFileQuarantined):
|
||||||
orig_metadata = {}
|
orig_metadata = {}
|
||||||
|
@@ -273,6 +273,12 @@ class TestObject(unittest2.TestCase):
|
|||||||
})
|
})
|
||||||
|
|
||||||
def test_if_none_match(self):
|
def test_if_none_match(self):
|
||||||
|
def delete(url, token, parsed, conn):
|
||||||
|
conn.request('DELETE', '%s/%s/%s' % (
|
||||||
|
parsed.path, self.container, 'if_none_match_test'), '',
|
||||||
|
{'X-Auth-Token': token})
|
||||||
|
return check_response(conn)
|
||||||
|
|
||||||
def put(url, token, parsed, conn):
|
def put(url, token, parsed, conn):
|
||||||
conn.request('PUT', '%s/%s/%s' % (
|
conn.request('PUT', '%s/%s/%s' % (
|
||||||
parsed.path, self.container, 'if_none_match_test'), '',
|
parsed.path, self.container, 'if_none_match_test'), '',
|
||||||
@@ -287,6 +293,13 @@ class TestObject(unittest2.TestCase):
|
|||||||
resp.read()
|
resp.read()
|
||||||
self.assertEqual(resp.status, 412)
|
self.assertEqual(resp.status, 412)
|
||||||
|
|
||||||
|
resp = retry(delete)
|
||||||
|
resp.read()
|
||||||
|
self.assertEqual(resp.status, 204)
|
||||||
|
resp = retry(put)
|
||||||
|
resp.read()
|
||||||
|
self.assertEqual(resp.status, 201)
|
||||||
|
|
||||||
def put(url, token, parsed, conn):
|
def put(url, token, parsed, conn):
|
||||||
conn.request('PUT', '%s/%s/%s' % (
|
conn.request('PUT', '%s/%s/%s' % (
|
||||||
parsed.path, self.container, 'if_none_match_test'), '',
|
parsed.path, self.container, 'if_none_match_test'), '',
|
||||||
|
Reference in New Issue
Block a user