openstack
/
keystone
Code Issues Proposed changes

register the config generator default hook with the right name 

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:49:31 -05:00
parent 529d08d8cd
commit 16768058d0
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
setup.cfg
View File

@ -176,7 +176,7 @@ oslo.config.opts =
keystone.notifications = keystone.notifications:list_opts
oslo.config.opts.defaults =
oslo.middleware = keystone.common.config:set_middleware_defaults
keystone = keystone.common.config:set_middleware_defaults
paste.filter_factory =
admin_token_auth = keystone.middleware:AdminTokenAuthMiddleware.factory