moved kerberose import to loaders

Author: Bilal Al

setup.py

@@ -23,7 +23,6 @@
     'requests',
     'pyyaml',
     'docopt>=0.6.2',
-    'requests-kerberos>=0.14.0'
     'enum34;python_version<"3.4"',
     'bloom-filter2>=2.0.0'
 ]
@@ -47,7 +46,8 @@
         'redis': ['redis>=2.10.5'],
         'uwsgi': ['uwsgi>=2.0.0'],
         'cpphash': ['mmh3cffi==0.2.1'],
-        'asyncio': ['aiohttp>=3.8.4', 'aiofiles>=23.1.0']
+        'asyncio': ['aiohttp>=3.8.4', 'aiofiles>=23.1.0'],
+        'kerberos': ['requests-kerberos>=0.14.0']
     },
     setup_requires=['pytest-runner', 'pluggy==1.0.0;python_version<"3.8"'],
     classifiers=[

splitio/api/client.py

@@ -5,7 +5,7 @@
 import abc
 import logging
 import json
-from requests_kerberos import HTTPKerberosAuth, OPTIONAL
+from splitio.optional.loaders import HTTPKerberosAuth, OPTIONAL
 
 from splitio.client.config import AuthenticateScheme
 from splitio.optional.loaders import aiohttp

splitio/optional/loaders.py

@@ -14,5 +14,17 @@ def missing_asyncio_dependencies(*_, **__):
     asyncio = missing_asyncio_dependencies
     aiofiles = missing_asyncio_dependencies
 
+try:
+    from requests_kerberos import HTTPKerberosAuth, OPTIONAL
+except ImportError:
+    def missing_auth_dependencies(*_, **__):
+        """Fail if missing dependencies are used."""
+        raise NotImplementedError(
+            'Missing kerberos auth dependency. '
+            'Please use `pip install splitio_client[kerberos]` to install the sdk with kerberos auth support'
+        )
+    HTTPKerberosAuth = missing_auth_dependencies
+    OPTIONAL = missing_auth_dependencies
+
 async def _anext(it):
     return await it.__anext__()

splitio/version.py

@@ -1 +1 @@
-__version__ = '10.0.1'
+__version__ = '10.1.0-rc1'

tests/api/test_httpclient.py

@@ -168,6 +168,7 @@ def test_authentication_scheme(self, mocker):
         get_mock.return_value = response_mock
         mocker.patch('splitio.api.client.requests.get', new=get_mock)
         httpclient = client.HttpClient(sdk_url='https://sdk.com', authentication_scheme=AuthenticateScheme.KERBEROS)
+        httpclient.set_telemetry_data("metric", mocker.Mock())
         response = httpclient.get('sdk', '/test1', 'some_api_key', {'param1': 123}, {'h1': 'abc'})
         call = mocker.call(
             'https://sdk.com/test1',
@@ -178,6 +179,7 @@ def test_authentication_scheme(self, mocker):
         )
 
         httpclient = client.HttpClient(sdk_url='https://sdk.com', authentication_scheme=AuthenticateScheme.KERBEROS, authentication_params=['bilal', 'split'])
+        httpclient.set_telemetry_data("metric", mocker.Mock())
         response = httpclient.get('sdk', '/test1', 'some_api_key', {'param1': 123}, {'h1': 'abc'})
 
     def test_telemetry(self, mocker):