Merge "Add "stores" to disallowed properties"

This commit is contained in:
Zuul 2020-08-05 21:30:03 +00:00 committed by Gerrit Code Review
commit a5bd812a5f
3 changed files with 14 additions and 1 deletions

View File

@ -729,7 +729,7 @@ class ImagesController(object):
class RequestDeserializer(wsgi.JSONRequestDeserializer):
_disallowed_properties = ('direct_url', 'self', 'file', 'schema')
_disallowed_properties = ('direct_url', 'self', 'file', 'schema', 'stores')
_readonly_properties = ('created_at', 'updated_at', 'status', 'checksum',
'size', 'virtual_size', 'direct_url', 'self',
'file', 'schema', 'id', 'os_hash_algo',

View File

@ -4361,6 +4361,13 @@ class TestImagesDeserializerNoAdditionalProperties(test_utils.BaseTestCase):
self.assertRaises(webob.exc.HTTPBadRequest,
self.deserializer.create, request)
def test_neg_create_with_stores(self):
self.config(allow_additional_image_properties=True)
request = unit_test_utils.get_fake_request()
request.body = jsonutils.dump_as_bytes({'stores': 'test'})
self.assertRaises(webob.exc.HTTPForbidden,
self.deserializer.create, request)
def test_update(self):
request = unit_test_utils.get_fake_request()
request.content_type = 'application/openstack-images-v2.1-json-patch'

View File

@ -0,0 +1,6 @@
---
fixes:
- |
Bug 1889676_: "stores" can be set as property breaking multistore indication of stores where the images are present
.. _1889676: https://bugs.launchpad.net/glance/+bug/1889676