feat: Additional context and breadcrumbs #390
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR:
src/main/context.ts
a littlecontext.app.app_start_time
ElectronEvents
integration toElectronBreadcrumbs
. We should use Sentry terminology to avoid confusion.BrowserWindow
andautoUpdater
eventsBrowserWindow
andWebContents
events, we now include theid
and window title as extra breadcrumb dataAdditionalContext
integration with all options enabled by default{ cpu, screen, memory, language }
cpu
addsprocessor_count
,cpu_description
andprocessor_frequency
tocontext.device
context.device.machine_arch = 'arm64'
if machine arch isarm64
and running in emulationscreen
addsscreen_resolution
andscreen_density
tocontext.device
memory
addsmemory_size
andfree_memory
tocontext.device
language
addslanguage
tocontext.device
All context in
src/main/context.ts
is not user configurable. TheAdditionalContext
integration allow this extra context to be configurable. The only downside to using an integration is that anyone using theElectronMinidump
uploader rather than the default sentry uploader will not get this additional context because those events don't go through the event processors.Closes #386
ElectronBreadcrumbs
configurationStop getting
app
events:Only get
show
andhide
events forBrowserWindow
Get all
app
events apart from those starting withremote-
: