RecursionError when there are log attributes assigned to None

[OS-joaocastilho]

Describe your environment:

Python version: 3.10

Steps to reproduce:

Inject attributes assigned to None into LogRecord in a LogRecordFactory.
(logging.LogRecord) record.custom_attribute = None

What is the expected behavior?
The SDK should handle the injection of attributes assigned to None into LogRecord without entering into a recursion loop.

What is the actual behavior?
The SDK enters a recursion loop, leading to a RecursionError, causing the service to die.

Additional context:

This issue occurs when injecting attributes assigned to None into the LogRecord in a LogRecordFactory that will be consumed by the SDK Logs.
The problem is specifically in the _clean_attribute method of the BoundedAttributes class, which generates a new logger.warning and leads to an infinite loop when the attribute is assigned as None.
While a workaround is possible on the service side by removing the None attribute, it is essential to address this issue in the SDK to prevent the logging module from causing service failures.
]()

[miikka]

If anyone else runs into this issue because they upgraded to Python 3.12 and now taskName attribute in log records is None: that specific case was addressed in #3557.

[hamzmu]

@OS-joaocastilho has this issue been resolved? if not can i pick it up.

[OS-joaocastilho]

@hamzmu it's not and we found out that this behaviour not only happens in None atributes, but in all types that aren't supported. You have defined the list of supported types in the same file.
Entering in a recursion Loop when the type is not supported does not seem a right behaviour for a logging module.

[lzchen]

It seems like the underlying issue is the fact that you have configured your telemetry collection to collect logs on the api/sdk itself, which is not the intended usage. The infinite recursion issue would exist in any case where the OpenTelemetry api/sdk records a log and you have configured your logging to track those logs as well. Our recommendation is to use namespaced loggers and attach the OpenTelemetry LoggingHandler only to the namespaced loggers you want to collect telemetry for.

[lzchen]

Closing due to inactivity.