Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Remove robots.txt and replace with <meta name=robots value=noindex> #2806

Closed
ianb opened this issue May 4, 2017 · 2 comments
Closed

Remove robots.txt and replace with <meta name=robots value=noindex> #2806

ianb opened this issue May 4, 2017 · 2 comments
Assignees

Comments

@ianb
Copy link
Contributor

ianb commented May 4, 2017

/robots.txt blocks all access by all robots, including friendly robots like the Twitter summarization bot (that turns a link into a nice card). I don't believe we can fix robots.txt unless we choose a small number of services to allow, and that doesn't align well with Mozilla's values. Plus it's annoying to maintain.

Our goal is to avoid shot pages being indexed by search engines. We can accomplish this with:

<meta name="robots" value="noindex" />

We should add that to shot pages. /shots is not public, and I don't believe any other pages are meaningfully private.

If we want to suppress indexing of images we'd have to add X-Robots-Tag: noindex to those requests. At this moment (maybe just due to oversight) we don't block indexing of images.

@ianb ianb self-assigned this May 4, 2017
@ianb
Copy link
Contributor Author

ianb commented May 4, 2017

Holding off on a PR for 1c6efd6 until @fmarier @jvehent @g-k or others chance for feedback

@ghost ghost added this to the Screenshots in 54 milestone May 5, 2017
ianb added a commit that referenced this issue May 15, 2018
This was intended to be fixed in #2806, but was only put in place for expired pages instead of all pages as intended.
jaredhirsch pushed a commit that referenced this issue May 15, 2018
This was intended to be fixed in #2806, but was only put in place for expired pages instead of all pages as intended.
ianb added a commit that referenced this issue May 15, 2018
This was intended to be fixed in #2806, but was only put in place for expired pages instead of all pages as intended.
testeaxeax pushed a commit to testeaxeax/screenshots that referenced this issue Jun 7, 2018
…ges (mozilla-services#4460)

This was intended to be fixed in mozilla-services#2806, but was only put in place for expired pages instead of all pages as intended.
chenba pushed a commit to chenba/screenshots that referenced this issue Jun 26, 2018
This was intended to be fixed in mozilla-services#2806, but was only put in place for expired pages instead of all pages as intended.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants