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
core(driver): add ' Chrome-Lighthouse' to useragent #7297
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -40,12 +40,12 @@ const DESKTOP_EMULATION_METRICS = { | |||
|
|||
const NEXUS5X_USERAGENT = { | |||
// eslint-disable-next-line max-len | |||
userAgent: 'Mozilla/5.0 (Linux; Android 6.0.1; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3694.0 Mobile Safari/537.36', | |||
userAgent: 'Mozilla/5.0 (Linux; Android 6.0.1; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3694.0 Mobile Safari/537.36 Chrome-Lighthouse', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so just sticking Chrome-Lighthouse
on the end is what PSI currently does? 😮
well TIL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Just making sure (seems ok if PSI is using this today), we're pretty sure this won't break most people's terrible UA sniffing code?
SURE! |
Summary
Adds 'Chrome-Lighthouse' to the UA for mobile and desktop so that Lighthouse and PSI have strongly consistent UAs.
Will come to PSI and web.dev on next update.
Related Issues/PRs
addresses: #7140