logtail-python 0.3.0__tar.gz → 0.3.2__tar.gz

{logtail_python-0.3.0/logtail_python.egg-info → logtail_python-0.3.2}/PKG-INFO

@@ -1,9 +1,9 @@
 Metadata-Version: 2.1
 Name: logtail-python
-Version: 0.3.0
+Version: 0.3.2
 Summary: Logtail.com client library
 Home-page: https://github.com/logtail/logtail-python
-Download-URL: https://github.com/logtail/logtail-python/tarball/0.3.0
+Download-URL: https://github.com/logtail/logtail-python/tarball/0.3.2
 Author: Logtail
 Author-email: hello@logtail.com
 License: ISC

{logtail_python-0.3.0 → logtail_python-0.3.2}/logtail/__init__.py

@@ -5,6 +5,6 @@ from .handler import LogtailHandler
 from .helpers import LogtailContext, DEFAULT_CONTEXT
 from .formatter import LogtailFormatter
 
-__version__ = '0.3.0'
+__version__ = '0.3.2'
 
 context = DEFAULT_CONTEXT

{logtail_python-0.3.0 → logtail_python-0.3.2}/logtail/frame.py

@@ -60,17 +60,28 @@ def _parse_custom_events(record, include_extra_attributes):
 def _remove_circular_dependencies(obj, memo=None):
     if memo is None:
         memo = set()
-    if id(obj) in memo:
+
+    # Skip immutable types, which can't contain circular dependencies
+    if isinstance(obj, (str, int, float, bool, type(None))):
+        return obj
+
+    # For mutable objects, check for circular references
+    obj_id = id(obj)
+    if obj_id in memo:
         return "<omitted circular reference>"
-    memo.add(id(obj))
+    memo.add(obj_id)
+
     if isinstance(obj, dict):
         new_dict = {}
         for key, value in obj.items():
-            new_dict[key] = _remove_circular_dependencies(value, memo)
+            new_dict[key] = _remove_circular_dependencies(value, memo.copy())
         return new_dict
     elif isinstance(obj, list):
-        new_list = [_remove_circular_dependencies(item, memo) for item in obj]
-        return new_list
+        return [_remove_circular_dependencies(item, memo.copy()) for item in obj]
+    elif isinstance(obj, tuple):
+        return tuple(_remove_circular_dependencies(item, memo.copy()) for item in obj)
+    elif isinstance(obj, set):
+        return {_remove_circular_dependencies(item, memo.copy()) for item in obj}
     else:
         return obj
 

{logtail_python-0.3.0 → logtail_python-0.3.2/logtail_python.egg-info}/PKG-INFO

@@ -1,9 +1,9 @@
 Metadata-Version: 2.1
 Name: logtail-python
-Version: 0.3.0
+Version: 0.3.2
 Summary: Logtail.com client library
 Home-page: https://github.com/logtail/logtail-python
-Download-URL: https://github.com/logtail/logtail-python/tarball/0.3.0
+Download-URL: https://github.com/logtail/logtail-python/tarball/0.3.2
 Author: Logtail
 Author-email: hello@logtail.com
 License: ISC

{logtail_python-0.3.0 → logtail_python-0.3.2}/setup.py

@@ -3,7 +3,7 @@ import os
 from setuptools import setup
 
 
-VERSION = '0.3.0'
+VERSION = '0.3.2'
 ROOT_DIR = os.path.dirname(__file__)
 
 REQUIREMENTS = [

{logtail_python-0.3.0 → logtail_python-0.3.2}/tests/test_handler.py

@@ -203,6 +203,47 @@ class TestLogtailHandler(unittest.TestCase):
         self.assertEqual(log_entry['data']['egg']['chicken'], "<omitted circular reference>")
         self.assertTrue(handler.pipe.empty())
 
+    @patch('logtail.handler.FlushWorker')
+    def test_can_have_multiple_instance_of_same_string_in_extra_data(self, MockWorker):
+        buffer_capacity = 1
+        handler = LogtailHandler(
+            source_token=self.source_token,
+            buffer_capacity=buffer_capacity
+        )
+
+        logger = logging.getLogger(__name__)
+        logger.handlers = []
+        logger.addHandler(handler)
+        test_string = 'this is a test string'
+        logger.info('hello', extra={'test1': test_string, 'test2': test_string})
+
+        log_entry = handler.pipe.get()
+
+        self.assertEqual(log_entry['message'], 'hello')
+        self.assertEqual(log_entry['test1'], 'this is a test string')
+        self.assertEqual(log_entry['test2'], 'this is a test string')
+        self.assertTrue(handler.pipe.empty())
+
+    @patch('logtail.handler.FlushWorker')
+    def test_can_have_multiple_instance_of_same_array_in_extra_data(self, MockWorker):
+        buffer_capacity = 1
+        handler = LogtailHandler(
+            source_token=self.source_token,
+            buffer_capacity=buffer_capacity
+        )
+
+        logger = logging.getLogger(__name__)
+        logger.handlers = []
+        logger.addHandler(handler)
+        test_array = ['this is a test string']
+        logger.info('hello', extra={'test1': test_array, 'test2': test_array})
+
+        log_entry = handler.pipe.get()
+
+        self.assertEqual(log_entry['message'], 'hello')
+        self.assertEqual(log_entry['test1'], ['this is a test string'])
+        self.assertEqual(log_entry['test2'], ['this is a test string'])
+        self.assertTrue(handler.pipe.empty())
 
     @patch('logtail.handler.FlushWorker')
     def test_can_send_circular_dependency_in_context(self, MockWorker):