add a method to build metadata headers

Author: mredolatti

splitio/api/__init__.py

@@ -12,3 +12,20 @@ def __init__(self, custom_message, status_code=None):
     def status_code(self):
         """Return HTTP status code."""
         return self._status_code
+
+
+def headers_from_metadata(sdk_metadata):
+    """
+    Generate a dict with headers required by data-recording API endpoints.
+
+    :param sdk_metadata: SDK Metadata object, generated at sdk initialization time.
+    :type sdk_metadata: splitio.client.util.SdkMetadata
+
+    :return: A dictionary with headers.
+    :rtype: dict
+    """
+    return {
+        'SplitSDKVersion': sdk_metadata.sdk_version,
+        'SplitSDKMachineIP': sdk_metadata.instance_ip,
+        'SplitSDKMachineName': sdk_metadata.instance_name
+    }

splitio/api/events.py

@@ -3,7 +3,7 @@
 
 from future.utils import raise_from
 
-from splitio.api import APIException
+from splitio.api import APIException, headers_from_metadata
 from splitio.api.client import HttpClientException
 
 
@@ -24,11 +24,7 @@ def __init__(self, http_client, apikey, sdk_metadata):
         self._logger = logging.getLogger(self.__class__.__name__)
         self._client = http_client
         self._apikey = apikey
-        self._metadata = {
-            'SplitSDKVersion': sdk_metadata.sdk_version,
-            'SplitSDKMachineIP': sdk_metadata.instance_ip,
-            'SplitSDKMachineName': sdk_metadata.instance_name
-        }
+        self._metadata = headers_from_metadata(sdk_metadata)
 
     @staticmethod
     def _build_bulk(events):

splitio/api/impressions.py

@@ -5,7 +5,7 @@
 
 from future.utils import raise_from
 
-from splitio.api import APIException
+from splitio.api import APIException, headers_from_metadata
 from splitio.api.client import HttpClientException
 
 
@@ -24,11 +24,7 @@ def __init__(self, client, apikey, sdk_metadata):
         self._logger = logging.getLogger(self.__class__.__name__)
         self._client = client
         self._apikey = apikey
-        self._metadata = {
-            'SplitSDKVersion': sdk_metadata.sdk_version,
-            'SplitSDKMachineIP': sdk_metadata.instance_ip,
-            'SplitSDKMachineName': sdk_metadata.instance_name
-        }
+        self._metadata = headers_from_metadata(sdk_metadata)
 
     @staticmethod
     def _build_bulk(impressions):

splitio/api/telemetry.py

@@ -4,7 +4,7 @@
 import six
 from future.utils import raise_from
 
-from splitio.api import APIException
+from splitio.api import APIException, headers_from_metadata
 from splitio.api.client import HttpClientException
 
 
@@ -25,12 +25,7 @@ def __init__(self, client, apikey, sdk_metadata):
         self._logger = logging.getLogger(self.__class__.__name__)
         self._client = client
         self._apikey = apikey
-        self._metadata = {
-            'SplitSDKVersion': sdk_metadata.sdk_version,
-            'SplitSDKMachineIP': sdk_metadata.instance_ip,
-            'SplitSDKMachineName': sdk_metadata.instance_name
-        }
-
+        self._metadata = headers_from_metadata(sdk_metadata)
     @staticmethod
     def _build_latencies(latencies):
         """

splitio/client/listener.py

@@ -51,8 +51,6 @@ def log_impression(self, impression, attributes=None):
         try:
             self.impression_listener.log_impression(data)
         except Exception as exc:  #pylint: disable=broad-except
-            import traceback
-            traceback.print_exc()
             raise_from(
                 ImpressionListenerException('Error in log_impression user\'s method is throwing exceptions'),
                 exc