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

Feature/robots dissalow on staging #740

Merged
merged 6 commits into from
May 19, 2015

Conversation

yez
Copy link

@yez yez commented May 5, 2015

This pull request addresses #634

A per-subdomain robots.txt pattern has been introduced. The default is to load the production robots.txt file (currently empty with no guidelines for search engines). Staging version of the robots.txt file prevents all scraping from robots on entire site.

Jake Yesbeck added 5 commits May 4, 2015 23:11
New route to /robots.txt and some specific logic to switch
on if the subdomain is staging or not.

Staging will not allow any crawlers at all, production does not
have any current limitations
Also assert that the response is the actual file contents
instead of mocking the method call
@coveralls
Copy link

Coverage Status

Coverage decreased (-58.66%) to 25.03% when pulling 7ebaa6a on yez:feature/robots-dissalow-on-staging into d7ef598 on Growstuff:dev.

@yez yez force-pushed the feature/robots-dissalow-on-staging branch from 7ebaa6a to c1ab161 Compare May 5, 2015 16:18
@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 83.78% when pulling c1ab161 on yez:feature/robots-dissalow-on-staging into d7ef598 on Growstuff:dev.

@tygriffin
Copy link
Contributor

This looks good to me. Note: even though we are generally deprecating controller specs, this use seems appropriate.

Skud added a commit that referenced this pull request May 19, 2015
@Skud Skud merged commit c47a1bc into Growstuff:dev May 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants