BUG: lazy logging incompatible with MultiEmailHandler?

[rderooy]

Describe the bug
An issue I have with the MultiEmailHandler is that my log messages are written to be compliant with lazy logging (pylint warning W1201).

This works fine with the other loggers I use, but breaks with RedMail MultiEmailHandler, where my messages are full of '%s' where variables need to be.

e.g. this breaks RedMail and remains as "%s":

LOG.info("message %s", variable)

But this works:

LOG.info("message " + variable)

To Reproduce
Steps to reproduce the behavior:

1. Use lazy logging function
2. observe email, the %s will not be substituted for the variable

Expected behavior
%s to be substituted for the variable, like with other logging functions. Or at least some explanation in the docs about this.

Screenshots
If applicable, add screenshots to help explain your problem.

Email provider:

• Email service: private
• Application to view the email: not relevant

Environment (please complete the following information if relevant):

• OS: Linux
• Python version: 3.11
• Red Mail version: 0.6.0

[rderooy]

Based on lack of activity (last commit was from March 2024 and that was just a simple doc fix), I'm starting to suspect this project is dead.

I'm still using the MultiEmailHandler, but I'm wondering if I need to start looking for something else.

In any case, I'm not using lazy-logging (%s logging), and here is more-or-less what I do to prevent pylint from complaining:

import logging
import logging.handlers
from redmail.log import MultiEmailHandler

# redmail MultiEmailHandler is incompatible with lazy-logging
# pylint: disable=W1201,W1202,W1203

LOG = logging.getLogger(__name__)
LOG.setLevel(logging.DEBUG)
smtp_server: str = "smtp.example.com"
sender_email: str =  "host@example.com"
receiver_email: list[str] = ["email-logs@example.com"]
subject_email: str = "{max_level_name} - logs "
email_handler = MultiEmailHandler(
    capacity=1024,
    **{
        "host": smtp_server,
        "port": 0,
        "sender": sender_email,
        "subject": subject_email,
        "receivers": receiver_email,
        "text": """Logging level:
                                    {% for record in records %}
                                    [{{ record.levelname }}] {{ record.msg }}
                                    {% endfor %}
                                """,
        "html": """
                                    <ul>
                                    {% for record in records %}
                                        <li>[{{ record.levelname }}] {{ record.msg }}</li>
                                    {% endfor %}
                                    </ul>
                                """,
    },  # type: ignore
)
email_handler.setLevel(logging.DEBUG)
LOG.addHandler(email_handler)

var: str = "var1"
LOG.info(f"variable: {var}")

And then I'm using f-style logging, instead of lazy-logging, which is not ideal, but at least the email logs are legible and not filled with %s for every variable.