Minecraft-Console-Client icon indicating copy to clipboard operation
Minecraft-Console-Client copied to clipboard

Add Sentry Error Tracking

Open breadbyte opened this issue 1 year ago • 2 comments

  • Added a notice that we're using Sentry when first generating the config file
  • Added a config option to disable Sentry crash logging

Currently sends information about the current .NET Runtime and the Operating System. Here is a screenshot of all the information that is sent to Sentry whenever a crash occurs:

Very long screenshot

Screenshot_11-2-2024_4480

Currently uses my personal DSN. Will need to change that to our DSN before merging.

breadbyte avatar Feb 10 '24 20:02 breadbyte

Haven't looked in to the code, so you might have already added this. But it would be a good idea to allow this only for binaries that we release here, because someone could modify the code and cause an error.

milutinke avatar Feb 10 '24 22:02 milutinke

We could put the DSN into repository secret/variable and populate the DSN during build-time, just like how we populate the build info. Correspondingly, the code would need to check the DSN string before initializing Sentry SDK.

ReinforceZwei avatar Feb 11 '24 08:02 ReinforceZwei