-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Update User-Agent strings for 2023 #765
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
Conversation
- Removed Firefox version earlier than Jun 1, 2022 - Removed Google Chrome version earlier than Jun 1, 2022 - Removed Microsoft Edge version earlier than Jun 1, 2022 - Removed Opera version earlier than Jan 1, 2023 - Updated Mac OS version strings (10.2.x ~ 13.3.x) - Extended support for Android 12/13 Fixes: gocolly#762
9fd6d6e
to
75c67a6
Compare
Thank you this is awesome! |
@asciimoo - Can you please review this and merge it. |
bump |
@asciimoo @WGH- I tag you here because I can see most PRs were merged/reviewed by you. Would you might to spare some time to review this PR, I believe many developer have his/her own fork to get User-agents list up to date, but it would be better to have a way to keep parent repository UA list updated. Before an automation is implemented, I think we should at least to keep it updated manually (at the moment, it's update manually so easily become outdated). |
Just made an update again to support more mobile devices. |
458b1d6
to
dc73588
Compare
Bump @asciimoo |
I personally think the code is too complicated though, just a static list of strings would be sufficient for the job. But let's be conservative for the time being. |
@WGH- I agree. I simply try to keep the old way before we find a way to keep it update automatically. Maybe it could be a static list generated by script and trigger monthly/quarterly with CI/CD job, that should be enough? |
I think the main problem is that it may generate some weird unlikely OS version/browser version combinations, which may tip off bot detection algorithms. For most purposes, just a few common UA strings should be more than enough. |
True 😂 That's the reason behind the scene why I introduce new UA strings for mobile. |
Fixes: #762