Logging mode can't be changed by accessor function

[eclectocrat]

Because logger instances are created as global variables and not updated later, you can end up calling logging_config.set_mode(...) to absolutely no effect.

For example:

import loguru, sys
from fastapi_websocket_rpc.logger import logging_config, LoggingModes, get_logger

logger = loguru.logger

# Enable fastapi_websocket_rpc logging with loguru
try:
    logging_config.set_mode(LoggingModes.LOGURU)
    logger.info("🔍 FastAPI WebSocket RPC logging enabled (using loguru)")
    web_rpc_logger = get_logger("RPC_CHANNEL")
    web_rpc_logger.remove()
    web_rpc_logger.add(
        sys.stderr,
        level="DEBUG",
        format="<level>{level: <8}</level> | <green>{name}</green>:<red>{function}</red>:<red>{line}</red> - <level>{message}</level>",
    )
    print("WEB_RPC_LOGGER", web_rpc_logger, type(web_rpc_logger))
    web_rpc_logger.debug("🔍 fastapi_websocket_rpc logger configured")
except ImportError:
    logger.warning("Could not import fastapi_websocket_rpc logging config")

This code will not change the already created logger in rpc_channel.py.

[orweis]

You are correct - but this isn't an issue as you can create the logger before importing fastapi_websokcet_rpc (and in particular before the RPC channel is imported) and configure it as you see fit.

If you need to change the logging mode dynamically, or you have another reason for needy to change this retroactively - you're welcome to open a PR to contribute the feature.

[eclectocrat]

I put that at the top of main.py, before any other imports, and the RPC_CHANNEL logger was still at the default logging mode. I'm not a python expert but perhaps the __init__.py brings in everything as soon as I want the logger utility? There may be some order of operations that would work, but the only fix I found was to use environment variables to set the logging mode.

This could at the very least use a bit more instruction in the README file.

Ultimately I abandoned using Loguru due to #57