diff options
author | wangxiyuan <wangxiyuan@huawei.com> | 2017-10-19 14:24:40 +0800 |
---|---|---|
committer | wangxiyuan <wangxiyuan@huawei.com> | 2018-02-22 02:19:06 +0000 |
commit | 8e9255d56d2da16a7fec4f57e28246bb5b9cb713 (patch) | |
tree | 4996d054f4037eb034acf32366216a535c0cb8a6 | |
parent | 316d1b4659e5e22ca16aecde118a304f5b261e3b (diff) | |
download | keystonemiddleware-8e9255d56d2da16a7fec4f57e28246bb5b9cb713.tar.gz |
Remove kwargs_to_fetch_token
kwargs_to_fetch_token was deprecated and should be
removed in Rocky now.
Change-Id: Ic247efb84c5133449ead6a9864bbd7748e5e74bd
3 files changed, 7 insertions, 19 deletions
diff --git a/keystonemiddleware/auth_token/__init__.py b/keystonemiddleware/auth_token/__init__.py index 6766818..98c47f5 100644 --- a/keystonemiddleware/auth_token/__init__.py +++ b/keystonemiddleware/auth_token/__init__.py @@ -215,7 +215,6 @@ object is stored. import binascii import copy import datetime -import warnings from keystoneauth1 import access from keystoneauth1 import adapter @@ -309,12 +308,6 @@ class BaseAuthProtocol(object): perform. """ - # NOTE(jamielennox): Default to True and remove in Queens. This is a - # compatibility flag to allow passing **kwargs to fetch_token(). This - # is basically to allow compatibility with keystone's override. We will - # assume all subclasses are ok with this being True in the Queens release. - kwargs_to_fetch_token = False - def __init__(self, app, log=_LOG, @@ -436,14 +429,7 @@ class BaseAuthProtocol(object): # NOTE(edmondsw): strip the token to remove any whitespace that may # have been passed along in the header per bug 1689468 token = token.strip() - if self.kwargs_to_fetch_token: - data = self.fetch_token(token, **kwargs) - else: - m = _('Implementations of auth_token must set ' - 'kwargs_to_fetch_token this will be the required and ' - 'assumed in Queens.') - warnings.warn(m) - data = self.fetch_token(token) + data = self.fetch_token(token, **kwargs) try: return data, access.create(body=data, auth_token=token) @@ -549,8 +535,6 @@ class AuthProtocol(BaseAuthProtocol): _SIGNING_CERT_FILE_NAME = 'signing_cert.pem' _SIGNING_CA_FILE_NAME = 'cacert.pem' - kwargs_to_fetch_token = True - def __init__(self, app, conf): log = logging.getLogger(conf.get('log_name', __name__)) log.info('Starting Keystone auth_token middleware') diff --git a/keystonemiddleware/tests/unit/auth_token/test_base_middleware.py b/keystonemiddleware/tests/unit/auth_token/test_base_middleware.py index 32e4338..a3c06bc 100644 --- a/keystonemiddleware/tests/unit/auth_token/test_base_middleware.py +++ b/keystonemiddleware/tests/unit/auth_token/test_base_middleware.py @@ -31,8 +31,6 @@ class FakeApp(object): class FetchingMiddleware(auth_token.BaseAuthProtocol): - kwargs_to_fetch_token = True - def __init__(self, app, token_dict={}, **kwargs): super(FetchingMiddleware, self).__init__(app, **kwargs) self.token_dict = token_dict diff --git a/releasenotes/notes/remove_kwargs_to_fetch_token-20e3451ed192ab6a.yaml b/releasenotes/notes/remove_kwargs_to_fetch_token-20e3451ed192ab6a.yaml new file mode 100644 index 0000000..147a77b --- /dev/null +++ b/releasenotes/notes/remove_kwargs_to_fetch_token-20e3451ed192ab6a.yaml @@ -0,0 +1,6 @@ +--- +other: + - > + The ``kwargs_to_fetch_token`` setting was removed from the + ``BaseAuthProtocol`` class. Implementations of auth_token now assume kwargs + will be passed to the ``fetch_token`` method. |