Skip to content
This repository was archived by the owner on Aug 26, 2022. It is now read-only.

bug 1199166 - disallow robots from comparing#3447

Merged
jwhitlock merged 1 commit intomasterfrom
dont-crawl-compare-1199166
Aug 27, 2015
Merged

bug 1199166 - disallow robots from comparing#3447
jwhitlock merged 1 commit intomasterfrom
dont-crawl-compare-1199166

Conversation

@groovecoder
Copy link
Contributor

Today's down-time incident was caused by Yahoo Slurp! crawling onto a bad and expensive compare transaction.

To help mitigate the risk of search crawlers triggering overload transactions, we want to add a number of new Disallow directives to our robots.txt, so we can expect well-behaved robots to stay away from them.

I used Google Webmaster Tools to verify that this robots.txt file will not disallow regular docs urls. I couldn't find equivalent webmaster tools for Yahoo, so I emailed Kev Needham (our Yahoo liaison) and set up a Google Analytics alert to email me if our traffic from Yahoo drops below our minimum observed threshold, just in case.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was this a suggested change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, suggested by Google webmaster tools

@groovecoder groovecoder force-pushed the dont-crawl-compare-1199166 branch 2 times, most recently from 21471a4 to 2637448 Compare August 27, 2015 15:20
@groovecoder groovecoder assigned jwhitlock and unassigned groovecoder Aug 27, 2015
@jwhitlock
Copy link
Contributor

+r would crawl again

jwhitlock added a commit that referenced this pull request Aug 27, 2015
bug 1199166 - disallow robots from comparing
@jwhitlock jwhitlock merged commit 7bcd8df into master Aug 27, 2015
@groovecoder
Copy link
Contributor Author

giphy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants