v2.1.x Chore - Refactor exception logging

[coderabbitai]

Description

This issue tracks the refactoring of redundant exception logging instances throughout the codebase.

Problem

Several places in the codebase use redundant exception objects in logging.exception() calls. Since logging.exception() already automatically includes the exception traceback and message, passing the exception string explicitly is redundant.

Example:

except Exception as _cause:  # pragma: no branch
    _LOGGER.debug(str(type(_cause)))
    _LOGGER.exception(str(_cause))  # Redundant - exception already included
    _LOGGER.debug(str((_cause.args)))

Recommended Fix

Replace redundant exception logging with a descriptive message:

except Exception as _cause:  # pragma: no branch
    _LOGGER.debug(str(type(_cause)))
    _LOGGER.exception("Exception occurred while processing")  # Better
    _LOGGER.debug(str((_cause.args)))

Or, if no additional context is needed:

except Exception as _cause:  # pragma: no branch
    _LOGGER.debug(str(type(_cause)))
    _LOGGER.exception("")  # Let the exception details speak for themselves
    _LOGGER.debug(str((_cause.args)))

Files to Refactor

• tests/__init__.py: Line ~197
• [Add additional instances found during implementation]

Related

• This issue was identified in PR Patch error style #385 (Patch error style #385 (comment))

[reactive-firewall]

Additional Occurrences

https://github.com/reactive-firewall/multicast/blob/19d38dbe14c4a5d85f36733aee05db4c509cb773/multicast/send.py#L399-L407

https://github.com/reactive-firewall/multicast/blob/19d38dbe14c4a5d85f36733aee05db4c509cb773/tests/run_selective.py#L66-L80

https://github.com/reactive-firewall/multicast/blob/19d38dbe14c4a5d85f36733aee05db4c509cb773/tests/__init__.py#L277-L291

https://github.com/reactive-firewall/multicast/blob/19d38dbe14c4a5d85f36733aee05db4c509cb773/tests/__init__.py#L150-L200

already including:

https://github.com/reactive-firewall/multicast/blob/19d38dbe14c4a5d85f36733aee05db4c509cb773/tests/__init__.py#L181-L185

---

Search of all uses of exception()

On GitHub.com:

https://github.com/search?q=%22.exception%28%22+repo%3Areactive-firewall%2Fmulticast&type=code

and locally:

git grep -F ".exception("