feat: add authentication_type method to authenticators

This commit adds a new method ('authentication_type()') to
the Authenticator base class, and each authenticator implementation
(subclasses of Authenticator).
The function will return a string indicating the
authenticator type ('basic', 'iam', etc.).
This brings the python core in line with the Go, Java and Node cores.

Author: padamstx

ibm_cloud_sdk_core/authenticators/authenticator.py

@@ -19,6 +19,16 @@
 
 class Authenticator(ABC):
     """This interface defines the common methods and constants associated with an Authenticator implementation."""
+
+    # Constants representing the various authenticator types.
+    AUTHTYPE_BASIC = 'basic'
+    AUTHTYPE_BEARERTOKEN = 'bearerToken'
+    AUTHTYPE_IAM = 'iam'
+    AUTHTYPE_CONTAINER = 'container'
+    AUTHTYPE_CP4D = 'cp4d'
+    AUTHTYPE_NOAUTH = 'noAuth'
+    AUTHTYPE_UNKNOWN = 'unknown'
+
     @abstractmethod
     def authenticate(self, req: dict) -> None:
         """Perform the necessary authentication steps for the specified request.
@@ -40,3 +50,8 @@ def validate(self) -> None:
         To be implemented by subclasses.
         """
         pass
+
+    # pylint: disable=R0201
+    def authentication_type(self) -> str:
+        """Returns the authenticator's type.  This method should be overridden by each authenticator implementation."""
+        return Authenticator.AUTHTYPE_UNKNOWN

ibm_cloud_sdk_core/authenticators/basic_authenticator.py

@@ -35,14 +35,16 @@ class BasicAuthenticator(Authenticator):
     Raises:
         ValueError: The username or password is not specified or contains invalid characters.
     """
-    authentication_type = 'basic'
 
     def __init__(self, username: str, password: str) -> None:
         self.username = username
         self.password = password
         self.validate()
         self.authorization_header = self.__construct_basic_auth_header()
 
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('basic')."""
+        return Authenticator.AUTHTYPE_BASIC
 
     def validate(self) -> None:
         """Validate username and password.
@@ -61,13 +63,12 @@ def validate(self) -> None:
                 'The username and password shouldn\'t start or end with curly brackets or quotes. '
                 'Please remove any surrounding {, }, or \" characters.')
 
-
     def __construct_basic_auth_header(self) -> str:
         authstring = "{0}:{1}".format(self.username, self.password)
-        base64_authorization = base64.b64encode(authstring.encode('utf-8')).decode('utf-8')
+        base64_authorization = base64.b64encode(
+            authstring.encode('utf-8')).decode('utf-8')
         return 'Basic {0}'.format(base64_authorization)
 
-
     def authenticate(self, req: Request) -> None:
         """Add basic authentication information to a request.
 

ibm_cloud_sdk_core/authenticators/bearer_token_authenticator.py

@@ -33,12 +33,15 @@ class BearerTokenAuthenticator(Authenticator):
     Raises:
         ValueError: Bearer token is none.
     """
-    authentication_type = 'bearerToken'
 
     def __init__(self, bearer_token: str) -> None:
         self.bearer_token = bearer_token
         self.validate()
 
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('bearertoken')."""
+        return Authenticator.AUTHTYPE_BEARERTOKEN
+
     def validate(self) -> None:
         """Validate the bearer token.
 

ibm_cloud_sdk_core/authenticators/container_authenticator.py

@@ -18,6 +18,7 @@
 
 from .iam_request_based_authenticator import IAMRequestBasedAuthenticator
 from ..token_managers.container_token_manager import ContainerTokenManager
+from .authenticator import Authenticator
 
 
 class ContainerAuthenticator(IAMRequestBasedAuthenticator):
@@ -62,7 +63,6 @@ class ContainerAuthenticator(IAMRequestBasedAuthenticator):
             ValueError: Neither of iam_profile_name or iam_profile_idk are set,
             or client_id, and/or client_secret are not valid for IAM token requests.
     """
-    authentication_type = 'container'
 
     def __init__(self,
                  cr_token_filename: Optional[str] = None,
@@ -86,6 +86,10 @@ def __init__(self,
 
         self.validate()
 
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('container')."""
+        return Authenticator.AUTHTYPE_CONTAINER
+
     def validate(self) -> None:
         """Validates the iam_profile_name, iam_profile_id, client_id, and client_secret for IAM token requests.
 

ibm_cloud_sdk_core/authenticators/cp4d_authenticator.py

@@ -50,7 +50,6 @@ class CloudPakForDataAuthenticator(Authenticator):
         TypeError: The `disable_ssl_verification` is not a bool.
         ValueError: The username, password/apikey, and/or url are not valid for CP4D token requests.
     """
-    authenticationdict = 'cp4d'
 
     def __init__(self,
                  username: str = None,
@@ -71,6 +70,10 @@ def __init__(self,
 
         self.validate()
 
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('cp4d')."""
+        return Authenticator.AUTHTYPE_CP4D
+
     def validate(self) -> None:
         """Validate username, password, and url for token requests.
 
@@ -85,7 +88,8 @@ def validate(self) -> None:
 
         if ((self.token_manager.password is None and self.token_manager.apikey is None)
                 or (self.token_manager.password is not None and self.token_manager.apikey is not None)):
-            raise ValueError('Exactly one of `apikey` or `password` must be specified.')
+            raise ValueError(
+                'Exactly one of `apikey` or `password` must be specified.')
 
         if self.token_manager.url is None:
             raise ValueError('The url shouldn\'t be None.')

ibm_cloud_sdk_core/authenticators/iam_authenticator.py

@@ -16,6 +16,7 @@
 
 from typing import Dict, Optional
 
+from .authenticator import Authenticator
 from .iam_request_based_authenticator import IAMRequestBasedAuthenticator
 from ..token_managers.iam_token_manager import IAMTokenManager
 from ..utils import has_bad_first_or_last_char
@@ -54,7 +55,6 @@ class IAMAuthenticator(IAMRequestBasedAuthenticator):
         TypeError: The `disable_ssl_verification` is not a bool.
         ValueError: The apikey, client_id, and/or client_secret are not valid for IAM token requests.
     """
-    authentication_type = 'iam'
 
     def __init__(self,
                  apikey: str,
@@ -77,6 +77,10 @@ def __init__(self,
 
         self.validate()
 
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('iam')."""
+        return Authenticator.AUTHTYPE_IAM
+
     def validate(self) -> None:
         """Validates the apikey, client_id, and client_secret for IAM token requests.
 

ibm_cloud_sdk_core/authenticators/no_auth_authenticator.py

@@ -19,7 +19,10 @@
 
 class NoAuthAuthenticator(Authenticator):
     """Performs no authentication."""
-    authentication_type = 'noAuth'
+
+    def authentication_type(self) -> str:
+        """Returns this authenticator's type ('noauth')."""
+        return Authenticator.AUTHTYPE_NOAUTH
 
     def validate(self) -> None:
         pass

ibm_cloud_sdk_core/get_authenticator.py

@@ -43,23 +43,28 @@ def get_authenticator_from_environment(service_name: str) -> Authenticator:
 def __construct_authenticator(config: dict) -> Authenticator:
     # Determine the authentication type if not specified explicitly.
     if config.get('AUTH_TYPE'):
-        auth_type = config.get('AUTH_TYPE').lower()
+        auth_type = config.get('AUTH_TYPE')
     elif config.get('AUTHTYPE'):
-        auth_type = config.get('AUTHTYPE').lower()
+        auth_type = config.get('AUTHTYPE')
     else:
+        # If authtype wasn't specified explicitly, then determine the default.
         # If the APIKEY property is specified, then it should be IAM, otherwise Container Auth.
-        auth_type = 'iam' if config.get('APIKEY') else 'container'
+        if config.get('APIKEY'):
+            auth_type = Authenticator.AUTHTYPE_IAM
+        else:
+            auth_type = Authenticator.AUTHTYPE_CONTAINER
 
+    auth_type = auth_type.lower()
     authenticator = None
 
-    if auth_type == 'basic':
+    if auth_type == Authenticator.AUTHTYPE_BASIC.lower():
         authenticator = BasicAuthenticator(
             username=config.get('USERNAME'),
             password=config.get('PASSWORD'))
-    elif auth_type == 'bearertoken':
+    elif auth_type == Authenticator.AUTHTYPE_BEARERTOKEN.lower():
         authenticator = BearerTokenAuthenticator(
             bearer_token=config.get('BEARER_TOKEN'))
-    elif auth_type == 'container':
+    elif auth_type == Authenticator.AUTHTYPE_CONTAINER.lower():
         authenticator = ContainerAuthenticator(
             cr_token_filename=config.get('CR_TOKEN_FILENAME'),
             iam_profile_name=config.get('IAM_PROFILE_NAME'),
@@ -70,14 +75,14 @@ def __construct_authenticator(config: dict) -> Authenticator:
             disable_ssl_verification=config.get(
                 'AUTH_DISABLE_SSL', 'false').lower() == 'true',
             scope=config.get('SCOPE'))
-    elif auth_type == 'cp4d':
+    elif auth_type == Authenticator.AUTHTYPE_CP4D.lower():
         authenticator = CloudPakForDataAuthenticator(
             username=config.get('USERNAME'),
             password=config.get('PASSWORD'),
             url=config.get('AUTH_URL'),
             apikey=config.get('APIKEY'),
             disable_ssl_verification=config.get('AUTH_DISABLE_SSL', 'false').lower() == 'true')
-    elif auth_type == 'iam' and config.get('APIKEY'):
+    elif auth_type == Authenticator.AUTHTYPE_IAM.lower() and config.get('APIKEY'):
         authenticator = IAMAuthenticator(
             apikey=config.get('APIKEY'),
             url=config.get('AUTH_URL'),
@@ -86,7 +91,7 @@ def __construct_authenticator(config: dict) -> Authenticator:
             disable_ssl_verification=config.get(
                 'AUTH_DISABLE_SSL', 'false').lower() == 'true',
             scope=config.get('SCOPE'))
-    elif auth_type == 'noauth':
+    elif auth_type == Authenticator.AUTHTYPE_NOAUTH.lower():
         authenticator = NoAuthAuthenticator()
 
     return authenticator

test/test_basic_authenticator.py

@@ -1,12 +1,13 @@
 # pylint: disable=missing-docstring
 import pytest
 
-from ibm_cloud_sdk_core.authenticators import BasicAuthenticator
+from ibm_cloud_sdk_core.authenticators import BasicAuthenticator, Authenticator
 
 
 def test_basic_authenticator():
     authenticator = BasicAuthenticator('my_username', 'my_password')
     assert authenticator is not None
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_BASIC
     assert authenticator.username == 'my_username'
     assert authenticator.password == 'my_password'
 

test/test_bearer_authenticator.py

@@ -1,12 +1,13 @@
 # pylint: disable=missing-docstring
 import pytest
 
-from ibm_cloud_sdk_core.authenticators import BearerTokenAuthenticator
+from ibm_cloud_sdk_core.authenticators import BearerTokenAuthenticator, Authenticator
 
 
 def test_bearer_authenticator():
     authenticator = BearerTokenAuthenticator('my_bearer_token')
     assert authenticator is not None
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_BEARERTOKEN
     assert authenticator.bearer_token == 'my_bearer_token'
 
     authenticator.set_bearer_token('james bond')

test/test_container_authenticator.py

@@ -1,12 +1,13 @@
 # pylint: disable=missing-docstring
 import pytest
 
-from ibm_cloud_sdk_core.authenticators import ContainerAuthenticator
+from ibm_cloud_sdk_core.authenticators import ContainerAuthenticator, Authenticator
 
 
 def test_container_authenticator():
     authenticator = ContainerAuthenticator(iam_profile_name='iam-user-123')
     assert authenticator is not None
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_CONTAINER
     assert authenticator.token_manager.cr_token_filename is None
     assert authenticator.token_manager.iam_profile_name == 'iam-user-123'
     assert authenticator.token_manager.iam_profile_id is None

test/test_cp4d_authenticator.py

@@ -5,13 +5,14 @@
 import pytest
 import responses
 
-from ibm_cloud_sdk_core.authenticators import CloudPakForDataAuthenticator
+from ibm_cloud_sdk_core.authenticators import CloudPakForDataAuthenticator, Authenticator
 
 
 def test_cp4d_authenticator():
     authenticator = CloudPakForDataAuthenticator(
         'my_username', 'my_password', 'http://my_url')
     assert authenticator is not None
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_CP4D
     assert authenticator.token_manager.url == 'http://my_url/v1/authorize'
     assert authenticator.token_manager.username == 'my_username'
     assert authenticator.token_manager.password == 'my_password'
@@ -40,7 +41,7 @@ def test_cp4d_authenticator():
 
 def test_disable_ssl_verification():
     authenticator = CloudPakForDataAuthenticator(
-            'my_username', 'my_password', 'http://my_url', disable_ssl_verification=True)
+        'my_username', 'my_password', 'http://my_url', disable_ssl_verification=True)
     assert authenticator.token_manager.disable_ssl_verification is True
 
     authenticator.set_disable_ssl_verification(False)
@@ -54,7 +55,7 @@ def test_invalid_disable_ssl_verification_type():
     assert str(err.value) == 'disable_ssl_verification must be a bool'
 
     authenticator = CloudPakForDataAuthenticator(
-            'my_username', 'my_password', 'http://my_url')
+        'my_username', 'my_password', 'http://my_url')
     assert authenticator.token_manager.disable_ssl_verification is False
 
     with pytest.raises(TypeError) as err:

test/test_iam_authenticator.py

@@ -5,12 +5,13 @@
 import pytest
 import responses
 
-from ibm_cloud_sdk_core.authenticators import IAMAuthenticator
+from ibm_cloud_sdk_core.authenticators import IAMAuthenticator, Authenticator
 
 
 def test_iam_authenticator():
     authenticator = IAMAuthenticator(apikey='my_apikey')
     assert authenticator is not None
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_IAM
     assert authenticator.token_manager.url == 'https://iam.cloud.ibm.com'
     assert authenticator.token_manager.client_id is None
     assert authenticator.token_manager.client_secret is None
@@ -46,7 +47,8 @@ def test_iam_authenticator():
 
 
 def test_disable_ssl_verification():
-    authenticator = IAMAuthenticator(apikey='my_apikey', disable_ssl_verification=True)
+    authenticator = IAMAuthenticator(
+        apikey='my_apikey', disable_ssl_verification=True)
     assert authenticator.token_manager.disable_ssl_verification is True
 
     authenticator.set_disable_ssl_verification(False)
@@ -55,7 +57,8 @@ def test_disable_ssl_verification():
 
 def test_invalid_disable_ssl_verification_type():
     with pytest.raises(TypeError) as err:
-        authenticator = IAMAuthenticator(apikey='my_apikey', disable_ssl_verification='True')
+        authenticator = IAMAuthenticator(
+            apikey='my_apikey', disable_ssl_verification='True')
     assert str(err.value) == 'disable_ssl_verification must be a bool'
 
     authenticator = IAMAuthenticator(apikey='my_apikey')

test/test_no_auth_authenticator.py

@@ -1,12 +1,12 @@
 # pylint: disable=missing-docstring
 
-from ibm_cloud_sdk_core.authenticators import NoAuthAuthenticator
+from ibm_cloud_sdk_core.authenticators import NoAuthAuthenticator, Authenticator
 
 
 def test_no_auth_authenticator():
     authenticator = NoAuthAuthenticator()
     assert authenticator is not None
-    assert authenticator.authentication_type == 'noAuth'
+    assert authenticator.authentication_type() == Authenticator.AUTHTYPE_NOAUTH
 
     authenticator.validate()