openstack/mistral
Code Issues Proposed changes

register the config generator default hook with the right name

Browse Source 
No config generator hooks should ever be registered with a name that
belongs to another project. In this case, using oslo.middleware.cors
means that *every other project* that loads the middleware gets this
application's defaults when the generator is run on a system with
everything installed (such as a dev box with devstack). Use the name
of the app instead, to ensure that the defaults are only set when this
app's sample config and documentation are being generated.

Change-Id: I6a8c7d44b9db9325003ff2fdb667b0ced7739e96
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This commit is contained in:
Doug Hellmann 2016-03-11 15:29:40 -05:00
parent 2e95865a67
commit 8a2d4b29d7
1 changed files with 1 additions and 1 deletions

2
setup.cfg

@ -38,7 +38,7 @@ oslo.config.opts =
mistral.config = mistral.config:list_opts
oslo.config.opts.defaults =
oslo.middleware.cors = mistral.config:set_cors_middleware_defaults
mistral.config = mistral.config:set_cors_middleware_defaults
tempest.test_plugins =
mistral_test = mistral_tempest_tests.plugin:MistralTempestPlugin