f3a49bf6e6
There is no separator Between request path and 'Accept' in the debug log. This patch adds white space separator between request path and 'Accept'. Change-Id: I82c553b50975a3c5898fa8ee6db10c796bd5987c
94 lines
3.2 KiB
Python
94 lines
3.2 KiB
Python
# Copyright (c) 2014 Mirantis, Inc.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
"""
|
|
A filter middleware that inspects the requested URI for a version string
|
|
and/or Accept headers and attempts to negotiate an API controller to
|
|
return
|
|
"""
|
|
|
|
from oslo_config import cfg
|
|
from oslo_log import log as logging
|
|
|
|
from murano.api import versions
|
|
from murano.common.i18n import _LW
|
|
from murano.common import wsgi
|
|
|
|
CONF = cfg.CONF
|
|
LOG = logging.getLogger(__name__)
|
|
|
|
|
|
class VersionNegotiationFilter(wsgi.Middleware):
|
|
@classmethod
|
|
def factory(cls, global_conf, **local_conf):
|
|
def filter(app):
|
|
return cls(app)
|
|
return filter
|
|
|
|
def __init__(self, app):
|
|
self.versions_app = versions.Controller()
|
|
super(VersionNegotiationFilter, self).__init__(app)
|
|
|
|
def process_request(self, req):
|
|
"""Try to find a version first in the accept header, then the URL."""
|
|
LOG.debug(("Determining version of request:{method} {path} "
|
|
"Accept: {accept}").format(method=req.method,
|
|
path=req.path,
|
|
accept=req.accept))
|
|
LOG.debug("Using url versioning")
|
|
# Remove version in url so it doesn't conflict later
|
|
req_version = self._pop_path_info(req)
|
|
|
|
try:
|
|
version = self._match_version_string(req_version)
|
|
except ValueError:
|
|
LOG.warning(_LW("Unknown version. Returning version choices."))
|
|
return self.versions_app
|
|
|
|
req.environ['api.version'] = version
|
|
req.path_info = ''.join(('/v', str(version), req.path_info))
|
|
LOG.debug("Matched version: v{version}".format(version=version))
|
|
LOG.debug('new path {path}'.format(path=req.path_info))
|
|
return None
|
|
|
|
def _match_version_string(self, subject):
|
|
"""Given a string, tries to match a major and/or
|
|
minor version number.
|
|
|
|
:param subject: The string to check
|
|
:returns version found in the subject
|
|
:raises ValueError if no acceptable version could be found
|
|
"""
|
|
if subject in ('v1',):
|
|
major_version = 1
|
|
else:
|
|
raise ValueError()
|
|
return major_version
|
|
|
|
def _pop_path_info(self, req):
|
|
"""'Pops' off the next segment of PATH_INFO, returns the popped
|
|
segment. Do NOT push it onto SCRIPT_NAME.
|
|
"""
|
|
path = req.path_info
|
|
if not path:
|
|
return None
|
|
while path.startswith('/'):
|
|
path = path[1:]
|
|
idx = path.find('/')
|
|
if idx == -1:
|
|
idx = len(path)
|
|
r = path[:idx]
|
|
req.path_info = path[idx:]
|
|
return r
|