Telegram notification error when log entry is > 4096 chars

Describe the bug When using the telegram notifier, if the log message is too long for a Telegram text message (> 4096 characters) a NotificationError with the message Notification errors: Bad Request: message is too long is raised.

To Reproduce

import logging

from notifiers.logging import NotificationHandler

message = "-" * 4097

logging.basicConfig(
    format="%(asctime)s - %(levelname)s - %(message)s")
defaults = {"token": bot_token, "chat_id": chat_id}
notification_handler = NotificationHandler("telegram", defaults=defaults)
notification_handler.setLevel(logging.ERROR)
logging.getLogger().addHandler(notification_handler)

logging.error(message)

Expected behavior Maybe the notification handler should split the log entry into parts each <= 4096 characters long. Then it can send the parts.

Additional context Python version: 3.8.10 OS: Windows 7 64-Bit

I don’t use the Telegram notifier (get_notifier("telegram")), but I also tried sending the same long message and instead of an exception being raised, the call ended successfully and no actual message was sent to the chat ID. But maybe another issue should be opened for that.

[liiight]

@allerter I given this some thought and research and I don’t think this issue should be handled by notifiers. I just supply a logger which redirects the logs to a provider, the responsibility for how to handle/cut the content should belong to the client. That being said, I limited the msg size for telegram to 4096 in #404, so now you would get a validation error and the message will not be sent

[a5r0n]

any updates?

i suggest to add some config to notifer, maybe split_long=True, and send_as_file=True?

i can put some PR for this