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

Sentry improvements #47

Open
9 tasks
Moneyl opened this issue Oct 7, 2023 · 0 comments
Open
9 tasks

Sentry improvements #47

Moneyl opened this issue Oct 7, 2023 · 0 comments

Comments

@Moneyl
Copy link
Contributor

Moneyl commented Oct 7, 2023

  • See if we can pass thread names to it. They show up in the VS debugger locally so the game must name them. Just not being sent to sentry.
  • Report non-crashing errors to sentry when useful. Examples:
    • When people get kicked or lose connection in MP. People still have some problems with this and need to rejoin or recreate the lobby.
    • When mempools/string pools being exceeded. These should also be logged locally and the player should be notified in the reconstructor overlay so the problem can be fixed quickly.
    • Look at all the spots that keen::debug::nativePrintArgs() is called. It's used in many cases like this. This will require a lot of manual hooks. Can't just hook that function since many of the calls have useless descriptions. Need to add in useful ones manually.
      • Report when mempools are exceeded
      • Report when string pools are exceeded
      • Report when arrays like base_array, farray, etc are exceeded
  • Include a list of loaded reconstructor plugins and their versions (will need to add version function export to plugins)
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

No branches or pull requests

1 participant