diff options
| author | Jenkins <jenkins@review.openstack.org> | 2013-06-27 22:51:05 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2013-06-27 22:51:05 +0000 |
| commit | 6c6d15ca8cc7376e0fff2d442b2cda8eebb324cf (patch) | |
| tree | 607c816dfa25d58afc5ec6fb454c7beee5cf1cf4 | |
| parent | 232630a20db04488e09ca28d507222c9123b6856 (diff) | |
| parent | f79ccf452b7a0cbf51c63adf259f46896586dc16 (diff) | |
| download | keystone-6c6d15ca8cc7376e0fff2d442b2cda8eebb324cf.tar.gz keystone-6c6d15ca8cc7376e0fff2d442b2cda8eebb324cf.tar.xz keystone-6c6d15ca8cc7376e0fff2d442b2cda8eebb324cf.zip | |
Merge "Do not raise NEW exceptions"
| -rw-r--r-- | keystone/auth/token_factory.py | 9 | ||||
| -rw-r--r-- | keystone/exception.py | 4 | ||||
| -rw-r--r-- | keystone/token/controllers.py | 9 |
3 files changed, 12 insertions, 10 deletions
diff --git a/keystone/auth/token_factory.py b/keystone/auth/token_factory.py index 64f945fa..fd44b637 100644 --- a/keystone/auth/token_factory.py +++ b/keystone/auth/token_factory.py @@ -17,6 +17,7 @@ """Token Factory""" import json +import sys import uuid import webob @@ -338,14 +339,14 @@ def create_token(context, auth_context, auth_info): token_data=token_data, trust_id=trust['id'] if trust else None) token_api.create_token(context, token_id, data) - except Exception as e: + except Exception: + exc_info = sys.exc_info() # an identical token may have been created already. # if so, return the token_data as it is also identical try: - token_api.get_token(context=context, - token_id=token_id) + token_api.get_token(context=context, token_id=token_id) except exception.TokenNotFound: - raise e + raise exc_info[0], exc_info[1], exc_info[2] return (token_id, token_data) diff --git a/keystone/exception.py b/keystone/exception.py index 39a08d4a..db5f5005 100644 --- a/keystone/exception.py +++ b/keystone/exception.py @@ -40,10 +40,10 @@ class Error(StandardError): try: message = self._build_message(message, **kwargs) - except KeyError as e: + except KeyError: # if you see this warning in your logs, please raise a bug report if _FATAL_EXCEPTION_FORMAT_ERRORS: - raise e + raise else: LOG.warning('missing exception kwargs (programmer error)') message = self.__doc__ diff --git a/keystone/token/controllers.py b/keystone/token/controllers.py index 68a3ebcd..9e19c9cb 100644 --- a/keystone/token/controllers.py +++ b/keystone/token/controllers.py @@ -1,4 +1,5 @@ import json +import sys import uuid from keystone.common import cms @@ -134,14 +135,14 @@ class Auth(controller.V2Controller): tenant=tenant_ref, metadata=metadata_ref, trust_id=trust_id)) - except Exception as e: + except Exception: + exc_info = sys.exc_info() # an identical token may have been created already. # if so, return the token_data as it is also identical try: - self.token_api.get_token(context=context, - token_id=token_id) + self.token_api.get_token(context=context, token_id=token_id) except exception.TokenNotFound: - raise e + raise exc_info[0], exc_info[1], exc_info[2] token_data['access']['token']['id'] = token_id |
