Skip to content

chore: make panic hook discord alert non-blocking#404

Open
Ocheretovich wants to merge 2 commits intogattaca-com:developfrom
Ocheretovich:develop
Open

chore: make panic hook discord alert non-blocking#404
Ocheretovich wants to merge 2 commits intogattaca-com:developfrom
Ocheretovich:develop

Conversation

@Ocheretovich
Copy link
Contributor

@Ocheretovich Ocheretovich commented Feb 26, 2026

Make Discord alert in panic hook non-blocking. Move blocking reqwest call into a separate thread and add timeout
to avoid potential hangs during shutdown.


Open with Devin

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 new potential issue.

View 4 additional findings in Devin Review.

Open in Devin Review

@Ocheretovich
Copy link
Contributor Author

CI failure seems unrelated to this PR (fails in postgres_db_service.rs on develop)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant