Merge "Add issued handlers to auth_ref and fixtures"
This commit is contained in:
@@ -120,6 +120,14 @@ class AccessInfo(dict):
|
|||||||
"""
|
"""
|
||||||
raise NotImplementedError()
|
raise NotImplementedError()
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued(self):
|
||||||
|
"""Returns the token issue time (as datetime object)
|
||||||
|
|
||||||
|
:returns: datetime
|
||||||
|
"""
|
||||||
|
raise NotImplementedError()
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def username(self):
|
def username(self):
|
||||||
"""Returns the username associated with the authentication request.
|
"""Returns the username associated with the authentication request.
|
||||||
@@ -363,6 +371,10 @@ class AccessInfoV2(AccessInfo):
|
|||||||
def expires(self):
|
def expires(self):
|
||||||
return timeutils.parse_isotime(self['token']['expires'])
|
return timeutils.parse_isotime(self['token']['expires'])
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued(self):
|
||||||
|
return timeutils.parse_isotime(self['token']['issued_at'])
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def username(self):
|
def username(self):
|
||||||
return self['user'].get('name', self['user'].get('username'))
|
return self['user'].get('name', self['user'].get('username'))
|
||||||
@@ -529,6 +541,10 @@ class AccessInfoV3(AccessInfo):
|
|||||||
def expires(self):
|
def expires(self):
|
||||||
return timeutils.parse_isotime(self['expires_at'])
|
return timeutils.parse_isotime(self['expires_at'])
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued(self):
|
||||||
|
return timeutils.parse_isotime(self['issued_at'])
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def user_id(self):
|
def user_id(self):
|
||||||
return self['user']['id']
|
return self['user']['id']
|
||||||
|
@@ -40,8 +40,8 @@ class Token(dict):
|
|||||||
that matter to them and not copy and paste sample.
|
that matter to them and not copy and paste sample.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def __init__(self, token_id=None,
|
def __init__(self, token_id=None, expires=None, issued=None,
|
||||||
expires=None, tenant_id=None, tenant_name=None, user_id=None,
|
tenant_id=None, tenant_name=None, user_id=None,
|
||||||
user_name=None):
|
user_name=None):
|
||||||
super(Token, self).__init__()
|
super(Token, self).__init__()
|
||||||
|
|
||||||
@@ -49,8 +49,16 @@ class Token(dict):
|
|||||||
self.user_id = user_id or uuid.uuid4().hex
|
self.user_id = user_id or uuid.uuid4().hex
|
||||||
self.user_name = user_name or uuid.uuid4().hex
|
self.user_name = user_name or uuid.uuid4().hex
|
||||||
|
|
||||||
|
if not issued:
|
||||||
|
issued = timeutils.utcnow() - datetime.timedelta(minutes=2)
|
||||||
if not expires:
|
if not expires:
|
||||||
expires = timeutils.utcnow() + datetime.timedelta(hours=1)
|
expires = issued + datetime.timedelta(hours=1)
|
||||||
|
|
||||||
|
try:
|
||||||
|
self.issued = issued
|
||||||
|
except (TypeError, AttributeError):
|
||||||
|
# issued should be able to be passed as a string so ignore
|
||||||
|
self.issued_str = issued
|
||||||
|
|
||||||
try:
|
try:
|
||||||
self.expires = expires
|
self.expires = expires
|
||||||
@@ -93,6 +101,22 @@ class Token(dict):
|
|||||||
def expires(self, value):
|
def expires(self, value):
|
||||||
self.expires_str = timeutils.isotime(value)
|
self.expires_str = timeutils.isotime(value)
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued_str(self):
|
||||||
|
return self._token['issued_at']
|
||||||
|
|
||||||
|
@issued_str.setter
|
||||||
|
def issued_str(self, value):
|
||||||
|
self._token['issued_at'] = value
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued(self):
|
||||||
|
return timeutils.parse_isotime(self.issued_str)
|
||||||
|
|
||||||
|
@issued.setter
|
||||||
|
def issued(self, value):
|
||||||
|
self.issued_str = timeutils.isotime(value)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def _user(self):
|
def _user(self):
|
||||||
return self.root.setdefault('user', {})
|
return self.root.setdefault('user', {})
|
||||||
|
@@ -54,7 +54,7 @@ class Token(dict):
|
|||||||
that matter to them and not copy and paste sample.
|
that matter to them and not copy and paste sample.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def __init__(self, expires=None, user_id=None, user_name=None,
|
def __init__(self, expires=None, issued=None, user_id=None, user_name=None,
|
||||||
user_domain_id=None, user_domain_name=None, methods=None,
|
user_domain_id=None, user_domain_name=None, methods=None,
|
||||||
project_id=None, project_name=None, project_domain_id=None,
|
project_id=None, project_name=None, project_domain_id=None,
|
||||||
project_domain_name=None, domain_id=None, domain_name=None,
|
project_domain_name=None, domain_id=None, domain_name=None,
|
||||||
@@ -71,8 +71,17 @@ class Token(dict):
|
|||||||
methods = ['password']
|
methods = ['password']
|
||||||
self.methods.extend(methods)
|
self.methods.extend(methods)
|
||||||
|
|
||||||
|
if not issued:
|
||||||
|
issued = timeutils.utcnow() - datetime.timedelta(minutes=2)
|
||||||
|
|
||||||
|
try:
|
||||||
|
self.issued = issued
|
||||||
|
except (TypeError, AttributeError):
|
||||||
|
# issued should be able to be passed as a string so ignore
|
||||||
|
self.issued_str = issued
|
||||||
|
|
||||||
if not expires:
|
if not expires:
|
||||||
expires = timeutils.utcnow() + datetime.timedelta(hours=1)
|
expires = self.issued + datetime.timedelta(hours=1)
|
||||||
|
|
||||||
try:
|
try:
|
||||||
self.expires = expires
|
self.expires = expires
|
||||||
@@ -115,7 +124,23 @@ class Token(dict):
|
|||||||
|
|
||||||
@expires.setter
|
@expires.setter
|
||||||
def expires(self, value):
|
def expires(self, value):
|
||||||
self.expires_str = timeutils.isotime(value)
|
self.expires_str = timeutils.isotime(value, subsecond=True)
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued_str(self):
|
||||||
|
return self.root.get('issued_at')
|
||||||
|
|
||||||
|
@issued_str.setter
|
||||||
|
def issued_str(self, value):
|
||||||
|
self.root['issued_at'] = value
|
||||||
|
|
||||||
|
@property
|
||||||
|
def issued(self):
|
||||||
|
return timeutils.parse_isotime(self.issued_str)
|
||||||
|
|
||||||
|
@issued.setter
|
||||||
|
def issued(self, value):
|
||||||
|
self.issued_str = timeutils.isotime(value, subsecond=True)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def _user(self):
|
def _user(self):
|
||||||
|
@@ -58,6 +58,7 @@ class AccessInfoTest(utils.TestCase, testresources.ResourcedTestCase):
|
|||||||
self.assertEqual(auth_ref.user_domain_name, 'Default')
|
self.assertEqual(auth_ref.user_domain_name, 'Default')
|
||||||
|
|
||||||
self.assertEqual(auth_ref.expires, token.expires)
|
self.assertEqual(auth_ref.expires, token.expires)
|
||||||
|
self.assertEqual(auth_ref.issued, token.issued)
|
||||||
|
|
||||||
def test_will_expire_soon(self):
|
def test_will_expire_soon(self):
|
||||||
token = client_fixtures.unscoped_token()
|
token = client_fixtures.unscoped_token()
|
||||||
|
@@ -61,6 +61,11 @@ class AccessInfoTest(utils.TestCase):
|
|||||||
|
|
||||||
self.assertEqual(auth_ref.expires, timeutils.parse_isotime(
|
self.assertEqual(auth_ref.expires, timeutils.parse_isotime(
|
||||||
UNSCOPED_TOKEN['token']['expires_at']))
|
UNSCOPED_TOKEN['token']['expires_at']))
|
||||||
|
self.assertEqual(auth_ref.issued, timeutils.parse_isotime(
|
||||||
|
UNSCOPED_TOKEN['token']['issued_at']))
|
||||||
|
|
||||||
|
self.assertEqual(auth_ref.expires, UNSCOPED_TOKEN.expires)
|
||||||
|
self.assertEqual(auth_ref.issued, UNSCOPED_TOKEN.issued)
|
||||||
|
|
||||||
def test_will_expire_soon(self):
|
def test_will_expire_soon(self):
|
||||||
expires = timeutils.utcnow() + datetime.timedelta(minutes=5)
|
expires = timeutils.utcnow() + datetime.timedelta(minutes=5)
|
||||||
|
Reference in New Issue
Block a user