Is this the best way to change the styling of only ERROR level log messages with RichHandler?

[sr105]

I want all ERROR messages to the console to be in red. I'm not talking about the level text, but the actual message. I'm completely new to rich. Looking at the code for RichHandler, it seems like a custom Highlighter is the best approach. Is there a better method?

I used a filter to change the highlighter on a given LogRecord depending on its level.

def _build_stdout_handler() -> logging.Handler:
    from rich.logging import RichHandler, Text
    from rich.highlighter import Highlighter, NullHighlighter

    class RedHighlighter(Highlighter):
        def highlight(self, text: Text) -> None:
            text.style = "bold red"

    def colored_error_filter(record: logging.LogRecord) -> (bool | logging.LogRecord):
        if record.levelno == logging.ERROR:
            record.highlighter = RedHighlighter()
            return record
        return True

    # RichHandler defaults to stdout
    # I don't want any other highlighting or special output.
    stdout_handler = RichHandler(show_time=False, show_level=False, show_path=False, highlighter=NullHighlighter())
    stdout_handler.addFilter(colored_error_filter)
    
    # the old handler that I'm replacing
    # stdout_handler = logging.StreamHandler(stream=sys.stdout)
    
    stdout_handler.setFormatter(logging.Formatter(fmt='%(message)s'))
    # stdout_handler.setLevel(logging.DEBUG)

    return stdout_handler

This is related to #3156.