RequestHandler refactor
Split out RequestHandler.__init__ into multiple functions Change-Id: I43925868044d32b665d401c53a284a803f726c8a
This commit is contained in:
@@ -107,10 +107,21 @@ class RequestHandler(object):
|
|||||||
service_providers.get(CONF, sp).enabled_services),
|
service_providers.get(CONF, sp).enabled_services),
|
||||||
CONF.service_providers
|
CONF.service_providers
|
||||||
)
|
)
|
||||||
|
self.mapping = None
|
||||||
|
|
||||||
|
self._handle_extension()
|
||||||
|
|
||||||
|
self._handle_action()
|
||||||
|
|
||||||
|
self._handle_sp_project()
|
||||||
|
|
||||||
|
self._log_request_to_proxy()
|
||||||
|
|
||||||
|
def _handle_extension(self):
|
||||||
for extension in self.extensions:
|
for extension in self.extensions:
|
||||||
extension.handle_request(self.details)
|
extension.handle_request(self.details)
|
||||||
|
|
||||||
|
def _handle_action(self):
|
||||||
if not self.details['version']:
|
if not self.details['version']:
|
||||||
if CONF.aggregation:
|
if CONF.aggregation:
|
||||||
# unversioned calls with no action
|
# unversioned calls with no action
|
||||||
@@ -123,6 +134,7 @@ class RequestHandler(object):
|
|||||||
# versioned calls with no action
|
# versioned calls with no action
|
||||||
abort(400)
|
abort(400)
|
||||||
|
|
||||||
|
def _handle_sp_project(self):
|
||||||
mapping = None
|
mapping = None
|
||||||
|
|
||||||
if self.details['resource_id']:
|
if self.details['resource_id']:
|
||||||
@@ -155,6 +167,7 @@ class RequestHandler(object):
|
|||||||
else:
|
else:
|
||||||
self._forward = self._forward
|
self._forward = self._forward
|
||||||
|
|
||||||
|
def _log_request_to_proxy(self):
|
||||||
LOG.info(format_for_log(title="Request to proxy",
|
LOG.info(format_for_log(title="Request to proxy",
|
||||||
method=self.details['method'],
|
method=self.details['method'],
|
||||||
url=self.details['path'],
|
url=self.details['path'],
|
||||||
|
|||||||
Reference in New Issue
Block a user