-
Notifications
You must be signed in to change notification settings - Fork 1
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
UA Financial Times library has disappeared #86
Comments
That is extremely unfortunate. They also removed https://github.com/Financial-Times/useragent_parser Maintaining a UA parser and/or normalizer is not something we have capacity for :( |
Here are the closest forks I've found: https://github.com/christianhaller3000/polyfill-useragent-normaliser and https://github.com/EthicalSecurity-Agency/Financial-Times_useragent_parser. We could compare these to what's on NPM to see what they are missing. |
It looks like https://github.com/EthicalSecurity-Agency/Financial-Times_useragent_parser has also been removed. I can try to find another recent fork, but I'm not sure what we're thinking of doing with it, given #92. How large of an effort do we think it will be to maintain a normalizer? It seems like the UA parser libraries already do a kind of normalization, which would then need to be mapped to our list of browsers and versions. As a test, I just took a list of user agents (link) and ran them through |
This library use @financial-times/polyfill-useragent-normaliser a library that was deprecated and removed his repository by financial times.
So maybe it should be change the library in the example and use another library or implement a first party UA normalizer.
The text was updated successfully, but these errors were encountered: