Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

Convert Invalid UA to metric #147

Closed
jrconlin opened this issue Jun 9, 2021 · 1 comment · Fixed by #153
Closed

Convert Invalid UA to metric #147

jrconlin opened this issue Jun 9, 2021 · 1 comment · Fixed by #153
Assignees
@jrconlin
Labels
2 Estimate - s - This is a small change with clearly defined parameters. good-first-issue Good for newcomers

Comments

@jrconlin
Copy link
Member

jrconlin commented Jun 9, 2021 •
edited by data-sync-user
Loading

https://sentry.prod.mozaws.net/operations/contile/issues/11582997/?environment=prod&referrer=alert_email

We will sometimes see spiders and other UA, because of course we will. We should convert these to metrics.

┆Issue is synchronized with this Jira Task
@jrconlin jrconlin added 2 Estimate - s - This is a small change with clearly defined parameters. good-first-issue Good for newcomers labels Jun 9, 2021
jrconlin added a commit that referenced this issue Jun 9, 2021
@jrconlin
feat: Do not report Invalid UAs to sentry
a048363 
Closes #147
@jrconlin
Copy link
Member Author

I can still see the value of having a discrete sentry error, but it shouldn't include the UA string since that makes each error unique.

jrconlin added a commit that referenced this issue Jun 10, 2021
@jrconlin
bug: make UA name an 'extra' value for Sentry errors.
1a519d0 
Closes #147
@jrconlin jrconlin mentioned this issue Jun 10, 2021
Merged
pjenvey pushed a commit that referenced this issue Jun 16, 2021
@jrconlin
bug: make UA name an 'extra' value for Sentry errors. (#153)
582c327 
Closes #147
Trinaa pushed a commit that referenced this issue Jun 13, 2022
@jrconlin
Delete position field from Tile model (#147)
008b337
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jrconlin jrconlin

Labels
2 Estimate - s - This is a small change with clearly defined parameters. good-first-issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bug: make UA name an 'extra' value for Sentry errors. mozilla-services/contile
1 participant
@jrconlin