openstack/nova
Code Issues Proposed changes

Merge "Add test coverage for API version headers in CORS"

Browse Source
This commit is contained in:
Zuul
2021-06-29 17:13:44 +00:00
committed by Gerrit Code Review
parent 25e2184849 d966546516
commit a70137f1d8
1 changed files with 28 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
nova/tests/functional/test_middleware.py
View File

@@ -122,3 +122,31 @@ class TestCORSMiddleware(api_sample_base.ApiSampleTestBaseV21):
self.assertEqual(response.status_code, self.exp_version_status)
self.assertNotIn('Access-Control-Allow-Origin', response.headers)
def test_api_version_request_headers(self):
# Verify allow_headers
headers = {
'Origin': 'http://valid.example.com',
'Access-Control-Request-Method': 'GET',
'Access-Control-Request-Headers':
'OpenStack-API-Version, X-OpenStack-Nova-API-Version'
}
response = self._do_options('', strip_version=True, headers=headers)
self.assertEqual(response.status_code, 200)
self.assertIn('Access-Control-Allow-Headers', response.headers)
self.assertIn('OpenStack-API-Version',
response.headers['Access-Control-Allow-Headers'])
self.assertIn('X-OpenStack-Nova-API-Version',
response.headers['Access-Control-Allow-Headers'])
# Verify expose_headers
headers = {'Origin': 'http://valid.example.com'}
response = self._do_get('', strip_version=True, headers=headers)
self.assertEqual(response.status_code, self.exp_version_status)
self.assertIn('Access-Control-Expose-Headers', response.headers)
self.assertIn('OpenStack-API-Version',
response.headers['Access-Control-Expose-Headers'])
self.assertIn('X-OpenStack-Nova-API-Version',
response.headers['Access-Control-Expose-Headers'])