Make changes

Author: DylanRussell

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py

@@ -120,7 +120,11 @@ def __init__(self, exporter: LogExporter):
 
     def on_emit(self, log_data: LogData):
         # Prevent entering recursive loop.
-        if any(item[2] == "on_emit" for item in traceback.extract_stack()):
+        if sum(
+            item.name == "on_emit"
+            and item.filename.endswith("export/__init__.py")
+            for item in traceback.extract_stack()
+        ) > 1:
             return
         if self._shutdown:
             _logger.warning("Processor is already shutdown, ignoring call")

opentelemetry-sdk/tests/logs/test_export.py

@@ -76,10 +76,10 @@ def export(self, batch: Sequence[LogData]):
         logger_provider.add_log_record_processor(
             SimpleLogRecordProcessor(exporter)
         )
-        logger = logging.getLogger("default_level")
-        logger.addHandler(LoggingHandler(logger_provider=logger_provider))
+        root_logger = logging.getLogger()
+        root_logger.addHandler(LoggingHandler(level=logging.NOTSET,logger_provider=logger_provider))
         # This would cause a max recursion depth exceeded error..
-        logger.warning("Something is wrong")
+        root_logger.warning("Something is wrong")
 
     def test_simple_log_record_processor_default_level(self):
         exporter = InMemoryLogExporter()