[TODO] Add an out-of-the box Sentry integration
webknjaz opened this issue · 4 comments
webknjaz commented
webknjaz commented
Pointers:
- https://github.com/python/miss-islington/pull/239/files
- https://github.com/underyx/aiohttp-sentry
- https://docs.sentry.io/platforms/python/aiohttp/
- Since we use a low-level HTTP web server API of aiohttp, we may need to go an extra mile to get this working
webknjaz commented
The initial implementation should just rely on the built-in support of SENTRY_DSN
env var in sentry_sdk
(sentry-python
) as it's also mirrored in probot: https://probot.github.io/docs/configuration/.
webknjaz commented
e741fb9 has the initial implementation but it needs more love in order to associate breadcrumbs with event handler calls...