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

Deal with error spikes caused by SWC email campaigns #848

Closed
travisbloom-cb opened this issue May 10, 2024 · 4 comments · Fixed by #919 or #1031
Closed

Deal with error spikes caused by SWC email campaigns #848

travisbloom-cb opened this issue May 10, 2024 · 4 comments · Fixed by #919 or #1031
Assignees
Labels
new sprint additions Label added to tickets that will be prioritized the next sprint.

Comments

@travisbloom-cb
Copy link
Contributor

travisbloom-cb commented May 10, 2024

Every time SWC team sends emails out to advocates we see a spike in error rates on the site. See this mixpanel view. Looking in sentry to correlate the error pages to actual exceptions yields examples like this. The error message is not helpful, but if you scroll down to the breadcrumbs you see we log the message "antifingerprint not defined yet. will try and handle event after its ready".

Investigating this error leads to this insight. It appear whats happening is that a percentage of advocates are using outlook, and their outlook app is scanning SWC when the email arrives.

We need to:

  • figure out how ignore these kinds of email bot scans in Sentry
  • figure out how to ignore these kinds of events in mixpanel

OR

  • figure out to get the error to not occur (which seems unlikely)
@travisbloom-cb travisbloom-cb added the new sprint additions Label added to tickets that will be prioritized the next sprint. label May 10, 2024
@ghost
Copy link

ghost commented May 10, 2024

According to this message and the docs from sentry, we should be able to ignore those errors on Sentry by just adding the following regex: /Non-Error promise rejection captured with value: Object Not Found/ to ignoreErrors inside Sentry.init.

This regex will filter out any string that contains the substring Non-Error promise rejection captured with value: Object Not Found, that is the immutable substring of the error message.

@travisbloom-cb
Copy link
Contributor Author

According to this message and the docs from sentry, we should be able to ignore those errors on Sentry by just adding the following regex: /Non-Error promise rejection captured with value: Object Not Found/ to ignoreErrors inside Sentry.init.

This regex will filter out any string that contains the substring Non-Error promise rejection captured with value: Object Not Found, that is the immutable substring of the error message.

Yup, we do this here, now just need to figure out how to ignore in mixpanel

@travisbloom-cb travisbloom-cb assigned ghost May 20, 2024
@ghost ghost mentioned this issue May 29, 2024
4 tasks
@ghost ghost closed this as completed in #919 May 29, 2024
@travisbloom-cb travisbloom-cb reopened this Jun 3, 2024
@travisbloom-cb
Copy link
Contributor Author

@paulovictor-cb reopening as it didn't prevent the errors from triggering as expected

@mhamendes
Copy link
Contributor

@olavoparno

@mhamendes mhamendes self-assigned this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new sprint additions Label added to tickets that will be prioritized the next sprint.
Projects
None yet
3 participants