From f58a73495f81dea700f229117f567bad93e9edaf Mon Sep 17 00:00:00 2001 From: kairat_kushaev <kkushaev@mirantis.com> Date: Mon, 4 Apr 2016 18:48:02 +0300 Subject: [PATCH] Remove unused skip_authentication decorator skip_authentication is not used as decorator for glanceclient methods. So this method can be safely removed from glance codebase because it is artifact from old implementation. Change-Id: I235b4c6b835c75266d8fae1bb603685aa17ad497 --- glanceclient/common/utils.py | 15 --------------- glanceclient/shell.py | 4 +--- 2 files changed, 1 insertion(+), 18 deletions(-) diff --git a/glanceclient/common/utils.py b/glanceclient/common/utils.py index c7dd4dd4..2723841c 100644 --- a/glanceclient/common/utils.py +++ b/glanceclient/common/utils.py @@ -246,21 +246,6 @@ def find_resource(manager, name_or_id): return matches[0] -def skip_authentication(f): - """Function decorator used to indicate a caller may be unauthenticated.""" - f.require_authentication = False - return f - - -def is_authentication_required(f): - """Checks to see if the function requires authentication. - - Use the skip_authentication decorator to indicate a caller may - skip the authentication step. - """ - return getattr(f, 'require_authentication', True) - - def env(*vars, **kwargs): """Search for the first defined of possibly many env vars. diff --git a/glanceclient/shell.py b/glanceclient/shell.py index 8f2a205c..a4564f63 100644 --- a/glanceclient/shell.py +++ b/glanceclient/shell.py @@ -428,9 +428,7 @@ class OpenStackImagesShell(object): endpoint = self._get_image_url(args) auth_token = args.os_auth_token - auth_req = (hasattr(args, 'func') and - utils.is_authentication_required(args.func)) - if not auth_req or (endpoint and auth_token): + if endpoint and auth_token: kwargs = { 'token': auth_token, 'insecure': args.insecure,