search

pieroxy / java-user-agent-detection

Some code to deduce an OS/Platform/Browser out of a user-agent string
53 stars 12 forks source link

Yandex is not a Baidu company #12

Closed mrDoctorWho closed 7 years ago

mrDoctorWho commented 7 years ago 
genericBotsBrandAndType.put("YandexAntivirus", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));
genericBotsBrandAndType.put("YandexFavicons", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));
genericBotsBrandAndType.put("YandexMedia", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));
genericBotsBrandAndType.put("YandexImages", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));
genericBotsBrandAndType.put("YandexImageResizer", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));
genericBotsBrandAndType.put("YandexBot", new Bot(Brand.BAIDU, BotFamily.CRAWLER, "Yandex Crawler", ""));

Your code suggests that Yandex Bot is a Baidu brand, but it isn't — https://yandex.com/support/search/robots/user-agent.html or https://wikipedia.org/wiki/Yandex.

pieroxy commented 7 years ago

Ah, ok, thanks. Will fix shortly.

pieroxy commented 7 years ago

Fixed in the latest commit. Will release shortly.