implement fake sentry service
Categories
(Socorro :: General, task, P2)
Tracking
(Not tracked)
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(1 file)
The existing code for the collector, processor, crontabber, and webapp all shut off sentry in the local dev and testing environments. The tests for sentry-related code don't actually test with the sentry client. Further, when there are problems with the error reporting code, the error reporting code is busted, so the errors with the error reporting code go unreported and unnoticed.
The problem with this is that we have nothing that tests with the sentry client, so when that changes, we have no idea.
We should implement a fake sentry service that we can run in the local dev and testing environments that captures and let's us access sentry error submission payloads, assert things against them, and help us debug sentry-related code.
|
Assignee | |
Comment 1•3 years ago
|
||
I toyed with this yesterday and threw together something. I was going to include it as part of Socorro because that's easier, but I need this for Tecken and Antenna, too, so I'm going to split it out as yet another project I'll maintain until the heat death of the sun.
|
|
Assignee | |
Comment 2•3 years ago
|
||
|
|
Assignee | |
Comment 3•3 years ago
|
||
willkg merged PR #5961: "bug 1748484: add fake sentry service" in 4520ff6.
This creates a new fakesentry service and changes the local dev and testing environments to use it for sentry errors.
It also adds an integration test that verifies that sentry is set up correctly and that at least part of the sanitization code is working.
|
|
Assignee | |
Comment 4•3 years ago
|
||
This was deployed just now in bug #1748856.
Description
•