diff options
author | Gage Hugo <gagehugo@gmail.com> | 2019-07-08 10:35:05 -0500 |
---|---|---|
committer | Gage Hugo <gagehugo@gmail.com> | 2019-11-27 10:52:23 -0600 |
commit | a6a3edb80ead7fa1c4587d40461bd68e3f501fe7 (patch) | |
tree | 139d7ca3bebb418f8d9d8a585bf08005c7353fb1 /keystonemiddleware/auth_token | |
parent | 19f2791082f906132c65770e50334648d8f0c6c0 (diff) | |
download | keystonemiddleware-a6a3edb80ead7fa1c4587d40461bd68e3f501fe7.tar.gz |
Remove v2.0 functionality
This change removes v2.0 functionality from
keystonemiddleware, as well as associated tests.
Partial-Bug: #1845539
Partial-Bug: #1777177
Change-Id: If47e90085d8a59c52fb23876dc329cd4f0b05ef0
Diffstat (limited to 'keystonemiddleware/auth_token')
-rw-r--r-- | keystonemiddleware/auth_token/_identity.py | 35 |
1 files changed, 7 insertions, 28 deletions
diff --git a/keystonemiddleware/auth_token/_identity.py b/keystonemiddleware/auth_token/_identity.py index 49bdf9a..53d3819 100644 --- a/keystonemiddleware/auth_token/_identity.py +++ b/keystonemiddleware/auth_token/_identity.py @@ -13,7 +13,6 @@ from keystoneauth1 import discover from keystoneauth1 import exceptions as ksa_exceptions from keystoneauth1 import plugin -from keystoneclient.v2_0 import client as v2_client from keystoneclient.v3 import client as v3_client from six.moves import urllib @@ -37,25 +36,6 @@ class _RequestStrategy(object): pass -class _V2RequestStrategy(_RequestStrategy): - - AUTH_VERSION = (2, 0) - - def __init__(self, adap, **kwargs): - super(_V2RequestStrategy, self).__init__(adap, **kwargs) - self._client = v2_client.Client(session=adap) - - def verify_token(self, token, allow_expired=False): - # NOTE(jamielennox): allow_expired is ignored on V2 - auth_ref = self._client.tokens.validate_access_info(token) - - if not auth_ref: - msg = _('Failed to fetch token data from identity server') - raise ksm_exceptions.InvalidToken(msg) - - return {'access': auth_ref} - - class _V3RequestStrategy(_RequestStrategy): AUTH_VERSION = (3, 0) @@ -81,7 +61,7 @@ class _V3RequestStrategy(_RequestStrategy): return {'token': auth_ref} -_REQUEST_STRATEGIES = [_V3RequestStrategy, _V2RequestStrategy] +_REQUEST_STRATEGIES = [_V3RequestStrategy] class IdentityServer(object): @@ -137,13 +117,12 @@ class IdentityServer(object): def _get_strategy_class(self): if self._requested_auth_version: - # A specific version was requested. - if discover.version_match(_V3RequestStrategy.AUTH_VERSION, - self._requested_auth_version): - return _V3RequestStrategy - - # The version isn't v3 so we don't know what to do. Just assume V2. - return _V2RequestStrategy + if not discover.version_match(_V3RequestStrategy.AUTH_VERSION, + self._requested_auth_interface): + self._LOG.info('A version other than v3 was requested: %s', + self._requested_auth_interface) + # Return v3, even if the request is unknown + return _V3RequestStrategy # Specific version was not requested then we fall through to # discovering available versions from the server |