logtail-python 0.3.1__tar.gz → 0.3.3__tar.gz

{logtail_python-0.3.1/logtail_python.egg-info → logtail_python-0.3.3}/PKG-INFO

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

{logtail_python-0.3.1 → logtail_python-0.3.3}/logtail/__init__.py

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

{logtail_python-0.3.1 → logtail_python-0.3.3}/logtail/frame.py

@@ -60,23 +60,28 @@ def _parse_custom_events(record, include_extra_attributes):
 def _remove_circular_dependencies(obj, memo=None):
     if memo is None:
         memo = set()
+
     # Skip immutable types, which can't contain circular dependencies
-    if isinstance(obj, (str, int, float, bool)):
+    if isinstance(obj, (str, int, float, bool, type(None))):
         return obj
-    if id(obj) in memo:
+
+    # 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):
-        return [_remove_circular_dependencies(item, memo) for item in obj]
+        return [_remove_circular_dependencies(item, memo.copy()) for item in obj]
     elif isinstance(obj, tuple):
-        return tuple(_remove_circular_dependencies(item, memo) for item in obj)
+        return tuple(_remove_circular_dependencies(item, memo.copy()) for item in obj)
     elif isinstance(obj, set):
-        return {_remove_circular_dependencies(item, memo) for item in obj}
+        return {_remove_circular_dependencies(item, memo.copy()) for item in obj}
     else:
         return obj
 

{logtail_python-0.3.1 → logtail_python-0.3.3}/logtail/handler.py

@@ -9,7 +9,7 @@ from .flusher import FlushWorker
 from .uploader import Uploader
 from .frame import create_frame
 
-DEFAULT_HOST = 'https://in.logs.betterstack.com'
+DEFAULT_HOST = 'in.logs.betterstack.com'
 DEFAULT_BUFFER_CAPACITY = 1000
 DEFAULT_FLUSH_INTERVAL = 1
 DEFAULT_CHECK_INTERVAL = 0.1
@@ -32,7 +32,10 @@ class LogtailHandler(logging.Handler):
                  level=logging.NOTSET):
         super(LogtailHandler, self).__init__(level=level)
         self.source_token = source_token
-        self.host = host
+        if host.startswith('https://') or host.startswith('http://'):
+            self.host = host
+        else:
+            self.host = "https://" + host
         self.context = context
         self.pipe = queue.Queue(maxsize=buffer_capacity)
         self.uploader = Uploader(self.source_token, self.host)

{logtail_python-0.3.1 → logtail_python-0.3.3/logtail_python.egg-info}/PKG-INFO

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

{logtail_python-0.3.1 → logtail_python-0.3.3}/setup.py

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

{logtail_python-0.3.1 → logtail_python-0.3.3}/tests/test_handler.py

@@ -19,7 +19,7 @@ class TestLogtailHandler(unittest.TestCase):
     def test_handler_creates_uploader_from_args(self, MockWorker):
         handler = LogtailHandler(source_token=self.source_token, host=self.host)
         self.assertEqual(handler.uploader.source_token, self.source_token)
-        self.assertEqual(handler.uploader.host, self.host)
+        self.assertEqual(handler.uploader.host, "https://" + self.host)
 
     @patch('logtail.handler.FlushWorker')
     def test_handler_creates_pipe_from_args(self, MockWorker):
@@ -224,6 +224,27 @@ class TestLogtailHandler(unittest.TestCase):
         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):
         buffer_capacity = 1