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
Firefox iOS is not marked as a modern brower #13126
Comments
Unfortunately, meteor is using a 6-year-old unmaintained package: https://www.npmjs.com/package/useragent This means that the database of existing user agents is not updated during the last 6 years :( There is a fork that pulls from an updated database I have not looked at the families of useragents but this might require a change in modern-browsers package to handle new browsers |
The package is actually able to identify it as firefoxIos but meteor just doesn't map it to anything so it's always legacy. |
In that case, seems the fix should be adding an alias to the correct family of useragents |
Yeah I'm able to fix it locally with this on my server:
This is not the first time I've had problems with a new browser being marked as legacy, so I feel like this experience is somewhat lacking as a developer. At this point, a new browser is more likely than not modern, so it's weird that be default every browser is defaulting to legacy instead of assuming modern. I also just happen to never want to server a legacy bundle anymore, but couldn't figure out how to do that. At a minimum it would be helpful to be able to log any browsers types that we don't have a mapping for, so I can react a bit quicker than I did with this one. |
Firefox ios gets parsed as firefoxIOS which is not in any of the aliases or anything in the modern browser:
This is something that does keep happening when a new browser starts existing or becomes more popular, is there a good way to get notified or log when some completely unexpected browser starts using our site?
For some other reason our legacy bundle is broken, so this lead to a subset of users being completely broken.
The text was updated successfully, but these errors were encountered: