Add support of loguru logger.

[heckad]

Please add support for loguru. I can make pull request but I don't know how to correctly integrate structure loggers.

[untitaker]

#228 is very relevant for this, it's possible you will run into similar issues here. We are happy to accept and merge PRs for those things, generally speaking

[jtfidje]

Hi! I just wanted to chime in with my solution here.
I solved this by following advice given in #228 as @untitaker pointed to :) ( Thanks @untitaker ! )

Basically, all I did was this:

from loguru import logger
from sentry_sdk.integrations.logging import (
    LoggingIntegration,
    BreadcrumbHandler,
    EventHandler,
)


.
.
.


if settings.sentry_dsn:
    _ = logger.add(
        BreadcrumbHandler(level=logging.DEBUG),
            diagnose=settings.loguru_diagnose,
            level=logging.DEBUG,
    )

    _ = logger.add(
        EventHandler(level=logging.ERROR),
            diagnose=settings.loguru_diagnose,
            level=logging.ERROR,
    )

    integrations = [
        LoggingIntegration(level=None, event_level=None),
    ]

I also work with a lib where I don't have much control over their use of the built-in logger, and so I have this snippet as well to intercept the built-in logger and route everything through loguru ( from the excellent loguru docs! ):

class InterceptHandler(logging.Handler):
    """Intercepts builtin logging messages and routes them to Loguru"""

    def emit(self, record):
        # Get corresponding Loguru level if it exists
        try:
            level = logger.level(record.levelname).name
        except ValueError:
            level = record.levelno

        # Find caller from where originated the logged message
        frame, depth = logging.currentframe(), 2
        while frame.f_code.co_filename == logging.__file__:
            frame = frame.f_back
            depth += 1
        logger.opt(depth=depth, exception=record.exc_info).log(
            level, record.getMessage()
        )


# Enable interceptor
logging.basicConfig(handlers=[InterceptHandler()], level=0)

[sandrovw64]

Hello guys, we been using loguru for a while and I believe it would be a good option to integrate with Sentry

[fratambot]

@jtfidje where the settings are coming from in your example ?

[jtfidje]

@jtfidje where the settings are coming from in your example ?

settings is an instance of a Settings-class that inherits from pydantic.BaseSettings:

from pydantic import BaseSettings
class Settings(BaseSettings):
    """My settings!"""
    .
    .
    .

I use this a lot to configure applications

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[kbakk]

I also work with a lib where I don't have much control over their use of the built-in logger, and so I have this snippet as well to intercept the built-in logger and route everything through loguru ( from the excellent loguru docs! ):

@jtfidje I'm seeing some issues with what I believe how InterceptHandler is working - I posted a description here (Delgan/loguru#344 (comment)). In short, I'm seeing sentry_sdk.integrations.logging:86 (logging) | Test info msg insted of __main__:42 (t) | Test info msg.