openstack/swift
Code Issues Proposed changes

Merge "Add CORS unit tests to base"

Browse Source
This commit is contained in:
Jenkins 2015-07-08 06:56:22 +00:00 committed by Gerrit Code Review
commit be757e7ae1
2 changed files with 34 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
swift/proxy/controllers/base.py
View File

@ -1617,7 +1617,8 @@ class Controller(object):
list_from_csv(req.headers['Access-Control-Request-Headers'])) list_from_csv(req.headers['Access-Control-Request-Headers']))
# Populate the response with the CORS preflight headers # Populate the response with the CORS preflight headers
if cors.get('allow_origin', '').strip() == '*': if cors.get('allow_origin') and \
cors.get('allow_origin').strip() == '*':
headers['access-control-allow-origin'] = '*' headers['access-control-allow-origin'] = '*'
else: else:
headers['access-control-allow-origin'] = req_origin_value headers['access-control-allow-origin'] = req_origin_value

33
test/unit/proxy/controllers/test_base.py
View File

@ -128,7 +128,7 @@ class FakeApp(object):
reason = RESPONSE_REASONS[response.status_int][0] reason = RESPONSE_REASONS[response.status_int][0]
start_response('%d %s' % (response.status_int, reason), start_response('%d %s' % (response.status_int, reason),
[(k, v) for k, v in response.headers.items()]) [(k, v) for k, v in response.headers.items()])
# It's a bit strnage, but the get_info cache stuff relies on the # It's a bit strange, but the get_info cache stuff relies on the
# app setting some keys in the environment as it makes requests # app setting some keys in the environment as it makes requests
# (in particular GETorHEAD_base) - so our fake does the same # (in particular GETorHEAD_base) - so our fake does the same
_set_info_cache(self, environ, response.account, _set_info_cache(self, environ, response.account,
@ -436,6 +436,37 @@ class TestFuncs(unittest.TestCase):
self.assertEquals(resp['length'], 5555) self.assertEquals(resp['length'], 5555)
self.assertEquals(resp['type'], 'text/plain') self.assertEquals(resp['type'], 'text/plain')
def test_options(self):
base = Controller(self.app)
base.account_name = 'a'
base.container_name = 'c'
origin = 'http://m.com'
self.app.cors_allow_origin = [origin]
req = Request.blank('/v1/a/c/o',
environ={'swift.cache': FakeCache()},
headers={'Origin': origin,
'Access-Control-Request-Method': 'GET'})
with patch('swift.proxy.controllers.base.'
'http_connect', fake_http_connect(200)):
resp = base.OPTIONS(req)
self.assertEqual(resp.status_int, 200)
def test_options_unauthorized(self):
base = Controller(self.app)
base.account_name = 'a'
base.container_name = 'c'
self.app.cors_allow_origin = ['http://NOT_IT']
req = Request.blank('/v1/a/c/o',
environ={'swift.cache': FakeCache()},
headers={'Origin': 'http://m.com',
'Access-Control-Request-Method': 'GET'})
with patch('swift.proxy.controllers.base.'
'http_connect', fake_http_connect(200)):
resp = base.OPTIONS(req)
self.assertEqual(resp.status_int, 401)
def test_headers_to_container_info_missing(self): def test_headers_to_container_info_missing(self):
resp = headers_to_container_info({}, 404) resp = headers_to_container_info({}, 404)
self.assertEquals(resp['status'], 404) self.assertEquals(resp['status'], 404)