RomanNum3ral
/
honcho
mirror of https://github.com/plastic-labs/honcho.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix Sentry IntegrityError (#119) 

* integrity error logger.error -> logger.warning

* add context comment for sentry + logging
This commit is contained in:
Rajat Ahuja 2025-06-16 17:56:53 -04:00 committed by GitHub
parent c22cc507b0
commit 4fab671e20
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/crud.py
View File

@ -124,7 +124,7 @@ async def create_app(db: AsyncSession, app: schemas.AppCreate) -> models.App:
return honcho_app
except IntegrityError as e:
await db.rollback()
logger.error(f"IntegrityError creating app with name '{app.name}': {str(e)}")
logger.warning(f"IntegrityError creating app with name '{app.name}': {str(e)}")
raise ConflictException(f"App with name '{app.name}' already exists") from e
@ -158,7 +158,7 @@ async def update_app(
return honcho_app
except IntegrityError as e:
await db.rollback()
logger.error(f"IntegrityError updating app {app_id}: {str(e)}")
logger.warning(f"IntegrityError updating app {app_id}: {str(e)}")
raise ConflictException(
"App update failed - unique constraint violation"
) from e

6
src/main.py
View File

@ -79,6 +79,12 @@ if SENTRY_ENABLED:
return event
# Sentry SDK's default behavior:
# - Captures INFO+ level logs as breadcrumbs
# - Captures ERROR+ level logs as Sentry events
#
# For custom log levels, use the LoggingIntegration class:
# sentry_sdk.init(..., integrations=[LoggingIntegration(level=logging.INFO, event_level=logging.ERROR)])
sentry_sdk.init(
dsn=os.getenv("SENTRY_DSN"),
traces_sample_rate=0.4,