From 13c2a15ae694c85b1b263a973ae00eaeef8a312d Mon Sep 17 00:00:00 2001 From: Lance Bragstad Date: Fri, 21 Jul 2017 17:06:49 +0000 Subject: [PATCH] Update comment about fetch token kwargs 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 --- keystonemiddleware/auth_token/__init__.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/keystonemiddleware/auth_token/__init__.py b/keystonemiddleware/auth_token/__init__.py index 1730df42..cc60cf72 100644 --- a/keystonemiddleware/auth_token/__init__.py +++ b/keystonemiddleware/auth_token/__init__.py @@ -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)