-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
ElectronBreadcrumbs
integration captures window titles by default
#459
Labels
Comments
While we work out how best to make this configurable you can remove window titles via the Sentry.init({
// other options...
beforeSend: (event) => {
for (const breadcrumb of event.breadcrumbs || []) {
if (breadcrumb?.data?.title) {
delete breadcrumb.data.title;
}
}
return event;
}
}) |
The obvious choice for a config option is the integration constructor: init({
dsn: '__DSN__',
integrations: [new ElectronBreadcrumbs({ includeWindowTitles: true })]
}) |
A fix for this has been released in 3.0.7 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Versions + Platform
@sentry/[email protected]
Description
cc @smeubank
The
ElectronBreadcrumbs
integration captures window titles by default and these may contain PII. This can currently be worked around by removing the integration but this level of data capture should be configurable.The text was updated successfully, but these errors were encountered: