From da40b42bc4b20b1cb1e1b1865b0c203bb932a792 Mon Sep 17 00:00:00 2001 From: Kaifeng Wang Date: Thu, 13 Sep 2018 15:39:06 +0800 Subject: [PATCH] Use set_defaults from cors cors provides set_defaults method about 2.5 year ago [1], let's call it a long debt gets paid. [1] https://review.openstack.org/#/c/285368 Change-Id: I566eb874d401772255b45e36135592bd736528e5 --- ironic_inspector/conf/opts.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ironic_inspector/conf/opts.py b/ironic_inspector/conf/opts.py index d480f1a33..69312a95b 100644 --- a/ironic_inspector/conf/opts.py +++ b/ironic_inspector/conf/opts.py @@ -39,9 +39,7 @@ def set_config_defaults(): def set_cors_middleware_defaults(): """Update default configuration options for oslo.middleware.""" - # TODO(krotscheck): Update with https://review.openstack.org/#/c/285368/ - cfg.set_defaults( - cors.CORS_OPTS, + cors.set_defaults( allow_headers=['X-Auth-Token', MIN_VERSION_HEADER, MAX_VERSION_HEADER,