openstack/keystone
Code Issues Proposed changes

Correct comment to not be driver-specific

Browse Source 
The comment mentioned Fernet tokens specifically, but the situation
is going to apply any time the driver doesn't persist the token.
The manager is common code and shouldn't depend on the driver.

Change-Id: I21c104b6256d9718c233d0f44d4c06c5a1ace6c9
This commit is contained in:
Brant Knudson 2015-09-23 15:57:57 -05:00
parent f311e7b428
commit f8fdabbd38
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
keystone/token/provider.py
View File

@ -232,10 +232,8 @@ class Manager(manager.Manager):
def validate_v3_token(self, token_id):
unique_id = utils.generate_unique_id(token_id)
# NOTE(lbragstad): Only go to persistent storage if we have a token to
# fetch from the backend. If the Fernet token provider is being used
# this step isn't necessary. The Fernet token reference is persisted in
# the token_id, so in this case set the token_ref as the identifier of
# the token.
# fetch from the backend (the driver persists the token). Otherwise
# the information about the token must be in the token id.
if not self._needs_persistence:
token_ref = token_id
else: