Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[back-end] Add Glitchtip error tracking #93

Merged
merged 2 commits into from
Nov 24, 2024
Merged

[back-end] Add Glitchtip error tracking #93

merged 2 commits into from
Nov 24, 2024

Conversation

rogup
Copy link
Contributor

@rogup rogup commented Nov 23, 2024

What? Why?

This will allow us to track uncaught exceptions and also 4xx and 5xx HTTP errors, viewable on the Glitchtip dashboard https://app.glitchtip.com/digital-commons-coop/issues?project=9203

(I included 4xx errors since we shouldn't really be hitting these if the FE is supplying the correct data)

It would be nice to notify Element devops channel but doesn't look like Glitchtip supports this yet. We could probably get it working eventually though, with some sort of workaround using webhooks or Matrix bots.

I've tested that this works on my local machine and the events are being picked up on the Glitchtip console

Deployment notes

Need to change export GLITCHTIP_KEY=<key> in the deploy.env to export FE_GLITCHTIP_KEY=<key>;export BE_GLITCHTIP_KEY=<key>; when deploying to the servers.

apps/back-end/src/errors.ts Outdated Show resolved Hide resolved
@wu-lee
Copy link
Contributor

wu-lee commented Nov 24, 2024

Apart from the very minor points in the comments, this is all good - will merge it myself as you suggest,

@wu-lee wu-lee force-pushed the glitchtip-back-end branch from c339750 to dbee63a Compare November 24, 2024 16:46
@wu-lee wu-lee merged commit 7ac5e22 into dev Nov 24, 2024
1 check passed
@wu-lee wu-lee deleted the glitchtip-back-end branch November 24, 2024 16:48
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.

2 participants