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

[Bug] WebVitals (e.g. CLS) being shown for non-Chromium browsers #928

Closed
1 task
paulb-elastic opened this issue Oct 28, 2020 · 7 comments
Closed
1 task
Assignees
Milestone

Comments

@paulb-elastic
Copy link

paulb-elastic commented Oct 28, 2020

It has been observed that WebVital values are being recorded from browsers that do not support WebVitals.
For example, we can see that there is a CLS score for a Safari on iPhone browser:

image

@hmdhk hmdhk added this to the Next milestone Oct 29, 2020
@paulb-elastic paulb-elastic modified the milestones: Next, 7.10 Oct 29, 2020
@vigneshshanmugam
Copy link
Member

I have verified with a test page on all safari versions from 12 - 14 on Mac OSX and was not able to reproduce the bug. Test was done through Saucelabs.

Safari even on tech preview does not support CLS metrics. So this is highly likely a bug in APM server user agent parser or it could from users who have extensions installed that alters the user agent header locally.

/cc @elastic/apm-server Team are you aware of any bugs in the the user agent parser that mistakes user agent header for Chrome as Safari in OSX?

@axw
Copy link
Member

axw commented Nov 3, 2020

What's highlighted there is the original, unparsed User-Agent header value. So I don't see how it could be a case of the server mistaking the user agent for Chrome.

Which version of APM Server is being used? One possibility is that data is getting mixed up between events. I have had no reason to believe this is happening previously, but there has been some upheaval in the decoding logic recently.

@simitt
Copy link

simitt commented Nov 3, 2020

@paulb-elastic with which APM Server version did you observe this behavior?

@paulb-elastic
Copy link
Author

@cachedout are you able to confirm what version of the APM Server the prod elastic.co data is being sent to, that ends up in the dev-next cluster?
cc @simitt

@simitt
Copy link

simitt commented Nov 3, 2020

I already received the information about the cluster in the mean time, thank you.

@cachedout
Copy link
Contributor

cachedout commented Nov 3, 2020

@cachedout are you able to confirm what version of the APM Server the prod elastic.co data is being sent to, that ends up in the dev-next cluster?

Traffic is mirroed to each of the APM servers associated with the clusters in question. So, for dev-next, the latest snapshot is deployed daily.

@paulb-elastic
Copy link
Author

Through a lot of investigation, it looks like these are coming from Chromium based clients that are changing the UA string, which is why we categorise them as another browser (e.g. Safari on iPhone), but the CLS is being able to be determined because it is actually a Chromium based browser. As such, closing this issue.

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

No branches or pull requests

6 participants