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

Opt-in Sentry instrumentation #1821

Open
loewenheim opened this issue Nov 14, 2023 · 2 comments
Open

Opt-in Sentry instrumentation #1821

loewenheim opened this issue Nov 14, 2023 · 2 comments

Comments

@loewenheim
Copy link
Contributor

loewenheim commented Nov 14, 2023

It would be useful to instrument sentry-cli with Sentry so that nothing runs on the user end by default, but we can ask them to temporarily switch it on in case of crashes.

More context

We do not currently instrument Sentry CLI with Sentry because users may not want us to know about how they are using Sentry CLI.

However, having Sentry instrumented in Sentry CLI might assist with debugging errors which are difficult for us to reproduce by allowing us to obtain from users a full stack trace of their error. #2189 was an attempt at allowing users to enable backtraces with the RUST_BACKTRACE=1 environment variable, but the change is insufficient because release builds don't include debug symbols, meaning that these stack traces are only really useful in a debug build.

To maintain user privacy, the Sentry instrumentation would be disabled by default, and would only be enabled when a certain environment variable (e.g. SENTRY_REPORT_ERROR) is set.

The CLI would then capture any errors that would crash the CLI and send them to Sentry, also outputting the event ID if this is possible.

Copy link

github-actions bot commented Dec 6, 2023

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@szokeasaurusrex
Copy link
Member

Perhaps, we could have the installer automatically prompt the user to ask them whether they would like their installation to be instrumented; we would then store this option in our configuration file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants