Skip to content

Create Inboundfilter for: @webkit-masked-url: (//hidden/) #7993

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

Closed
HazAT opened this issue Apr 28, 2023 Discussed in #5875 · 4 comments · Fixed by getsentry/relay#2088
Closed

Create Inboundfilter for: @webkit-masked-url: (//hidden/) #7993

HazAT opened this issue Apr 28, 2023 Discussed in #5875 · 4 comments · Fixed by getsentry/relay#2088
Assignees

Comments

@HazAT
Copy link
Member

HazAT commented Apr 28, 2023

See title, let's consider adding a default inboundfilter in Sentry to filter out these unhelpful/unactionable errors.

Discussed in #5875

Originally posted by Shramkoweb October 3, 2022
Hello. I began to see such errors in my project:

//hidden/ in y@webkit-masked-url: at line 2:2930760
//hidden/ in fromAny@webkit-masked-url: at line 2:2931890
//hidden/ in _handleResponsePromise@webkit-masked-url: at line 2:1144366
//hidden/ in _handleDispatchResponse@webkit-masked-url: at line 2:1144141
//hidden/ in _handleMessage@webkit-masked-url: at line 2:1143837
//hidden/ in @webkit-masked-url: at line 2:1143638
//hidden/ in emit@webkit-masked-url: at line 2339:17
//hidden/ in @webkit-masked-url: at line 2138:26

I can't understand what it is, why the sentry throws them. It looks like a code problem in Safari itself

@mydea
Copy link
Member

mydea commented May 2, 2023

So after checking in again with @lforst , I think what we want to do here is not add a new config/rule, but add this to the filters:browser-extensions rule we already have. Which means nothing really to do on our end, just change stuff in relay. I'll check with relay folks!

@mydea
Copy link
Member

mydea commented May 10, 2023

ref: #5875 (reply in thread) cc @HazAT @lforst can one of you follow up with the guy, as I am on PTO the next 2 weeks!

@lforst
Copy link
Member

lforst commented May 15, 2023

@mydea
Copy link
Member

mydea commented May 31, 2023

See also: https://sentry.sentry.io/issues/?project=1267915&project=4505153985052672&project=155735&project=6528548&project=4503972821204992&project=5953206&project=1886021&project=4504522681745408&project=6418660&project=6109199&project=1473210&project=1297621&project=5480435&project=76439&project=1466766&project=5738630&project=1492949&project=15626&project=282218&project=4503944281522176&project=4504120166776832&project=5266138&project=2783252&project=6619666&project=1297620&project=1492261&project=5246761&project=4505234376359936&project=221969&project=5496745&project=6356259&project=2783478&project=4504255174606848&project=4505031352713216&project=37617&project=5637584&project=5600888&project=4505160011087872&project=4505160012660736&project=4505148785885184&project=4504922584317952&project=4504922570424320&project=5662316&project=2450415&project=239368&project=4504753500979200&project=5613292&project=47606&project=5420004&project=11276&project=4504096424656896&project=4505001575776256&project=1505606&project=4505001573875712&project=4505041934155776&project=4505166992048128&project=4504933470633984&project=2721840&project=4857230&project=6681819&project=1492835&project=1489174&project=4505159881719808&project=4505207125901312&project=4503944114667520&project=5493568&project=4505070337589248&project=4504798982897664&project=4504130088468480&project=4505048029986816&project=5941975&project=1932911&project=4504041335357440&project=1492842&project=5711442&project=6520058&project=3487317&project=5641555&project=185747&project=133215&project=6380506&project=4504407171465216&project=4504890362298368&project=1292798&project=6183838&project=6416536&project=6627632&project=230989&project=1192882&project=54785&project=5898904&project=5905698&project=5683165&project=5683166&project=5723503&project=1492057&project=5899451&project=5903949&project=5726700&project=4504373448540160&project=162676&project=5645511&project=4504281272942592&project=4503892896251904&project=6690737&project=4504811881824256&project=6520785&project=5992455&project=300688&project=4505245339418624&project=191772&project=191841&project=1455593&project=1460611&project=1280593&project=1371326&project=5953213&project=6637965&project=4504923040710656&project=4846280&project=1419836&project=1360617&project=4505115731623936&project=4504763976581120&project=6178942&project=6301746&project=5324467&project=5350637&project=6424467&project=6681679&project=2053674&project=1297627&project=4504877012221952&project=5250619&project=5250630&project=5250627&project=200821&project=5250626&query=webkit&referrer=issue-list&statsPeriod=90d

TBS1996 added a commit to getsentry/relay that referenced this issue Jun 5, 2023
The added regex pattern should filter out the webkit issues linked from
here:
getsentry/sentry-javascript#7993 (comment)

The source is likely from safari extensions.

references: getsentry/sentry-javascript#7993
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants