-
Notifications
You must be signed in to change notification settings - Fork 129
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome.
with sentry we need to be extra careful not to send private and identifiable information so we won't have any privacy problems.
I don't think that's an issue at all. Data on sentry are not available in public access, so I don't see any problem in pushing data there. |
It is, for GDPR reasons. The less personal data travel throughout all the systems, the better. Otherwise it'll be very complicated. |
Kudos, SonarCloud Quality Gate passed! |
The only data sent to sentry are defined in sentry.js file, and are ids (+locale, but I don't think it's sensitive data). |
This pull request has been automatically locked since there |
This implements sentry for browser.
As a lot of our code is in javascript/vue, it's important to follow issues in there.
Sentry provides an implementation for vue, but it's not vue3 ready yet.
Activation of performance monitoring in the browser can be done by settings
SENTRY_TRACES_SAMPLE_RATE
> 0SENTRY_DEFAULT_PII
environment variable to enable Default Personally Identifiable Information featureYou fool, don't forget these steps: