Previously, something like logger.info("%s %s", "text", "123456789") would cause a logger error
since the mask_pii logic would replace the LogRecord's args tuple with the string '("text", "*****")',
which would break the log interpolation logic. This PR fixes that bug and adds some regression tests.
Testing
Added tests that failed before the fix and pass after the fix
Ticket
Resolves #154
Changes
Context
Previously, something like
logger.info("%s %s", "text", "123456789")would cause a logger error since the mask_pii logic would replace the LogRecord'sargstuple with the string '("text", "*****")', which would break the log interpolation logic. This PR fixes that bug and adds some regression tests.Testing
Added tests that failed before the fix and pass after the fix