openstack/keystonemiddleware
Code Issues Proposed changes

Update comment about fetch token kwargs

Browse Source 
Jamie had a comment explain the requirement of fetching tokens
with kwargs and the token. This was suppose to be required in Pike
but it was missed. This commit updates the comment to be relevant for
the Queens release instead.

Change-Id: Iaa2c3fb02e76a87865a4ae7f06c4e86cc5b9b991
This commit is contained in:
Lance Bragstad 2017-07-21 17:06:49 +00:00
parent 4a72cd6c3b
commit 13c2a15ae6
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
keystonemiddleware/auth_token/__init__.py
View File

@ -306,10 +306,10 @@ class BaseAuthProtocol(object):
perform.
"""
# NOTE(jamielennox): Default to True and remove in Pike. This is a
# 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 Pike release.
# assume all subclasses are ok with this being True in the Queens release.
kwargs_to_fetch_token = False
def __init__(self,
@ -435,7 +435,7 @@ class BaseAuthProtocol(object):
else:
m = _('Implementations of auth_token must set '
'kwargs_to_fetch_token this will be the required and '
'assumed in Pike.')
'assumed in Queens.')
warnings.warn(m)
data = self.fetch_token(token)