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.

Sign up for GitHub

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

Jump to bottom

Removes corvus in favor of sentry and analytics client #3891

Merged
merged 6 commits into from Jan 16, 2023
Merged

Removes corvus in favor of sentry and analytics client #3891

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
10caf8f
Removes corvus in favor of sentry and analytics client
otaviojacobi Nov 29, 2022
41fca03
Removes corvus in favor of sentry and analytics client
otaviojacobi Nov 29, 2022
f3f7ecb
Unifying sentry reports in a single project
aethernet Jan 6, 2023
2b728d3
Remove personal path on etcher
otaviojacobi Jan 6, 2023
6c417e3
patch: Sentry fix path
aethernet Jan 6, 2023
86d43a5
Anonymizes all paths before sending
otaviojacobi Jan 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
patch: Sentry fix path
  • Loading branch information
@aethernet @otaviojacobi
aethernet authored and otaviojacobi committed Jan 12, 2023
commit 6c417e35a13873cd95d25f42a819de3750cdf65d
2 changes: 1 addition & 1 deletion lib/gui/app/modules/analytics.ts
View file
Open in desktop
Expand Up @@ -23,7 +23,7 @@ import * as packageJSON from '../../../../package.json';

const clearUserPath = (filename: string): string => {
const generatedFile = filename.split('generated').reverse()[0];
return `generated${generatedFile}`;
return generatedFile !== filename ? `generated${generatedFile}` : filename;
};

export const anonymizeData = (
Expand Down