Merge pull request #31 from Jany26/sentry-setup

Configuring Sentry

Author: Jany26

README.md

@@ -51,6 +51,8 @@ to the value in their `Authorization` header.
 
 For example `Authorization: Bearer secret-123`.
 
+Additionally, for Sentry error and performance monitoring, `LD_PACKIT_INTERFACE_SENTRY_DSN` environment variable has to be set.
+
 ## Run the container
 
 Images are published to quay.io. If it isn't available, or if you want

pyproject.toml

@@ -11,6 +11,7 @@ dependencies = [
     "fastapi>=0.120.2",
     "fedora-messaging>=3.8.0",
     "gunicorn>=23.0.0",
+    "sentry_sdk>=2.17.0,<3.0.0",
     "httpx>=0.28.1",
     "pydantic>=2.12.3",
     "ruff>=0.14.2",

src/logdetective_packit/main.py

@@ -9,6 +9,7 @@
 
 from fastapi import FastAPI, Depends, HTTPException, status
 from fastapi.security import HTTPAuthorizationCredentials, HTTPBearer
+import sentry_sdk
 
 from httpx import AsyncClient, HTTPStatusError
 from fedora_messaging.api import publish, Message
@@ -33,6 +34,11 @@
 
 http_bearer = HTTPBearer()
 
+# Set the LD_PACKIT_INTERFACE_SENTRY_DSN env variable beforehand
+sentry_sdk.init(
+    dsn=os.environ.get("LD_PACKIT_INTERFACE_SENTRY_DSN")
+)
+
 app = FastAPI(title="LogDetectivePackit", version=version("logdetective-packit"))
 
 # Setup logging for fedora-messaging