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

General: Please add the following bots to jetpack #1402

Closed
nitinthewiz opened this issue Dec 11, 2014 · 3 comments

Comments

@nitinthewiz
Copy link

commented Dec 11, 2014

Hi,

I'm not sure how to do a pull request on this repo, since I'm not able to create a branch here. But I just want you folks to extend the list of bots that jetpack recognizes. To do that, please extend the class.jetpack-user-agent.php file with the following array -

$bot_agents = array(
            'alexa', 'altavista', 'ask jeeves', 'attentio', 'baiduspider', 'bingbot', 'chtml generic', 'crawler', 'fastmobilecrawl',
            'feedfetcher-google', 'firefly', 'froogle', 'gigabot', 'googlebot', 'googlebot-mobile', 'heritrix', 'ia_archiver', 'irlbot',
            'infoseek', 'jumpbot', 'lycos', 'mediapartners', 'mediobot', 'motionbot', 'msnbot', 'mshots', 'openbot',
            'pss-webkit-request',
            'pythumbnail', 'scooter', 'slurp', 'snapbot', 'spider', 'taptubot', 'technoratisnoop',
            'teoma', 'twiceler', 'yahooseeker', 'yahooysmcm', 'yammybot', 'ahrefsbot', 'pingdom.com_bot', 'kraken', 'yandexbot', 
            'twitterbot', 'tweetmemebot', 'openhosebot', 'queryseekerspider', 'linkdexbot', 'grokkit-crawler',
            'livelapbot', 'germcrawler', 'domaintunocrawler', 'grapeshotcrawler', 'cloudflare-alwaysonline',
        );

Notes on this edit are on my blog here.

Thanks!
Nitin

@nitinthewiz nitinthewiz changed the title Please add the following bots to jetpack General: Please add the following bots to jetpack Dec 11, 2014

@enejb enejb added this to the vFuture milestone Dec 12, 2014

@enejb

This comment has been minimized.

Copy link
Member

commented Dec 12, 2014

Hi nitinthewiz,

Thanks for reporting the issue and the blog post.
The list of bots is very helpful as well.

To learn how to create a pull request you can follow these steps.
https://help.github.com/articles/creating-a-pull-request/
I think you mentioned that you created a fork. Then you are half way there. :)

In the mean time I create a PR that will allow users to add their own list of bots.
You can find it here. #1403

@nitinthewiz

This comment has been minimized.

Copy link
Author

commented Dec 12, 2014

@enejb Thanks for your help. I believe I've found the right doc for fork+pull requests (https://help.github.com/articles/using-pull-requests/#before-you-begin)
and will use it next time.

Meanwhile, your filter method is definitely the better way forward and I can't wait to use it in the next jetpack version. Thanks! 👍

@jeherve jeherve added General and removed Stats labels Dec 12, 2014

@jeherve

This comment has been minimized.

Copy link
Member

commented Dec 12, 2014

@nitinthewiz You can also follow the steps here to create a Pull Request for Jetpack ;)
http://jetpack.me/contribute/#patch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.