search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.04k stars 4.19k forks source link

Microsoft Edge for iOS is detected as Mobile Safari #25420

Closed padinko closed 3 years ago

padinko commented 3 years ago

Micosoft Edge for iOS has user agent like:

Mozilla/5.0 (iPhone; CPU iPhone OS 14_4 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 EdgiOS/46.2.0 Mobile/15E148 Safari/605.1.15

with EdgiOS/version part, but sentry is not detecting this and showing Mobile Safari as a browser

lobsterkatie commented 3 years ago

We use the ua-parser package (which relies on data from uap-core) in our backend to translate from user agent string to browser name and version. If there's a mismatch, they're the ones who will be able to fix it.

Can you please create an issue in the uap-core repo? Once they fix their detection, we can update to use the new version.

(I'm going to move this to our main getsentry/sentry repo, since that's what will need to be updated to fix this once ua-parser makes their change.)

github-actions[bot] commented 3 years ago

Sorry, friend. As far as this ol' bot can tell, your issue does not properly use one of this repo's available issue templates. Please try again, if you like. (And if I'm confused, please let us know. 😬)

Did you see the memo about this?

(log)

lobsterkatie commented 3 years ago

@padinko - sorry, forgot our bots wouldn't accept the issue as is. Regardless, it's probably a good idea to create a new issue for the package update once ua-parser has done their fix, as this will get auto-closed after a certain amount of time even if I re-open it now.

padinko commented 3 years ago

ok, they already have issue I will watch it and create new issue on fix