openstack
/
deb-python-oslo.middleware
Code Issues Proposed changes

Remove deprecated oslo.subdomain from CORS opts 

Defining configuration under cors.* blocks is deprecated. However, we
advertise [cors.subdomain] as a CORS section, and still throw a warning
if that section (that we list) is used in config files.

This was causing warnings in the keystone unit tests output, since the
generated config file does contain both of those listed sections [2].

[1] https://github.com/openstack/oslo.middleware/blob/master/oslo_middleware/cors.py#L200-L205
[2] https://github.com/openstack/keystone/blob/master/etc/keystone.conf.sample#L587-L616

Change-Id: Ice114bf1fdceda7bf122f5fd6b98fc88a85c8ff3
This commit is contained in:
Samuel de Medeiros Queiroz 2017-06-19 16:54:58 -04:00
parent 3d65053741
commit bc7d4b8be8
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
oslo_middleware/opts.py
View File

@ -132,7 +132,6 @@ def list_opts_cors():
"""
return [
('cors', copy.deepcopy(cors.CORS_OPTS)),
('cors.subdomain', copy.deepcopy(cors.CORS_OPTS))
]