Correct QueueEmitHandler to only emit if the config key exists. (#22)

* only add the webhook handler if the config key exists.

* add webhook avatar

* resolve import

Author: AbstractUmbra

config.template.toml

@@ -16,6 +16,7 @@ dsn = 'postgres://pythonistabot:pythonistabot@database:5432/pythonistabot' # ass
 # 10 = DEBUG
 [LOGGING]
 webhook_url = ""  # optional
+webhook_avatar_url = "" # optional
 level = 20
 
 [SNEKBOX] # optional

core/utils/logging.py

@@ -7,6 +7,8 @@
 
 from discord.utils import _ColourFormatter as ColourFormatter, stream_supports_colour  # type: ignore # shh, I need it
 
+import core
+
 
 if TYPE_CHECKING:
     from typing_extensions import Self
@@ -62,6 +64,7 @@ def __enter__(self: Self) -> Self:
             if stream_supports_colour(stream_handler):
                 stream_handler.setFormatter(ColourFormatter())
             self.log.addHandler(stream_handler)
+        if core.CONFIG["LOGGING"].get("webhook_url"):
             self.log.addHandler(QueueEmitHandler(self.bot))
 
         return self

modules/logging.py

@@ -61,7 +61,11 @@ async def logging_loop(self) -> None:
 
         message = textwrap.shorten(f"{emoji} {format_dt(dt)}\n{to_log.message}", width=1990)
 
-        await self.webhook.send(message, username="PythonistaBot Logging")
+        await self.webhook.send(
+            message,
+            username="PythonistaBot Logging",
+            avatar_url=core.CONFIG["LOGGING"].get("webhook_avatar_url"),
+        )
 
 
 async def setup(bot: core.Bot) -> None:

types_/config.py

@@ -17,6 +17,7 @@ class Database(TypedDict):
 
 class Logging(TypedDict):
     webhook_url: NotRequired[str]
+    webhook_avatar_url: NotRequired[str]
     level: int