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

Flagged use of navigator.userAgent #1670

Open
aaron-613 opened this issue Aug 11, 2023 · 2 comments
Open

Flagged use of navigator.userAgent #1670

aaron-613 opened this issue Aug 11, 2023 · 2 comments

Comments

@aaron-613
Copy link

Hi there. My Chrome browser is complaining about the following:

Audit usage of navigator.userAgent, navigator.appVersion, and navigator.platform
A page or script is accessing at least one of navigator.userAgent, navigator.appVersion, and navigator.platform. Starting in Chrome 101, the amount of information available in the User Agent string will be reduced.
To fix this issue, replace the usage of navigator.userAgent, navigator.appVersion, and navigator.platform with feature detection, progressive enhancement, or migrate to navigator.userAgentData.

navigator.userAgent seems to appear in my minified version of MQTT.js. Checked the most recent version (5.0.2) and still there. I don't really mind, just flagging for your interest.

Thanks! And keep up the good work..!

@robertsLando
Copy link
Member

Wondering where that comes from as there is no navigator.userAgent in the code. I think it comes from some polifilly. Thanks for pointing this out

Copy link

This is an automated message to let you know that this issue has
gone 365 days without any activity. In order to ensure that we work
on issues that still matter, this issue will be closed in 14 days.

If this issue is still important, you can simply comment with a
"bump" to keep it open.

Thank you for your contribution.

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

2 participants