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

doc.rust-lang.org/robots.txt doesn't match src/doc/robots.txt #68677

Closed
carols10cents opened this issue Jan 30, 2020 · 4 comments · Fixed by #68678
Closed

doc.rust-lang.org/robots.txt doesn't match src/doc/robots.txt #68677

carols10cents opened this issue Jan 30, 2020 · 4 comments · Fixed by #68678
Labels
C-bug Category: This is a bug. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue.

Comments

@carols10cents
Copy link
Member

The intention in #59213 was to add robots.txt to what gets uploaded to doc.rust-lang.org, but it appears that either the manual step of copying this over for a release hasn't been happening, or that the automation was never done, or the automation isn't working.

The contents of src/doc/robots.txt doesn't match what I'm currently seeing on https://doc.rust-lang.org/robots.txt:

User-agent: *
Disallow: /0.3/
Disallow: /0.4/
Disallow: /0.5/
Disallow: /0.6/
Disallow: /0.7/
Disallow: /0.8/
Disallow: /0.9/
Disallow: /0.10/
Disallow: /0.11.0/
Disallow: /0.12.0/
Disallow: /1.0.0-alpha/
Disallow: /1.0.0-alpha.2/
Disallow: /1.0.0-beta/
Disallow: /1.0.0-beta.2/
Disallow: /1.0.0-beta.3/
Disallow: /1.0.0-beta.4/
Disallow: /1.0.0-beta.5/

(lines 20-27 in the repo are not present on the live site)

Thanks!

@carols10cents carols10cents added T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Jan 30, 2020
@jonas-schievink jonas-schievink added the C-bug Category: This is a bug. label Jan 30, 2020
@bors bors closed this as completed in 95df2bc Feb 3, 2020
@kornelski
Copy link
Contributor

Still not up to date after 1.42 release: https://doc.rust-lang.org/robots.txt

@Mark-Simulacrum
Copy link
Member

AFAICT, the relevant change has not yet made it into stable, but is in beta and nightly. I expect that means that we should get the robots.txt file populated appropriately in roughly 6 weeks. Closing this as fixed for now, if it's still wrong after 1.43 is released we can reopen (though ideally on rust-central-station, and not here, as remaining work should happen there).

@steveklabnik
Copy link
Member

Closing this as fixed for now, if it's still wrong after 1.43 is released we can reopen (though ideally on rust-central-station, and not here, as remaining work should happen there).

It seems like this fix didn't work after 1.43; I'm not as familiar with rust-central-station. What's the procedure for moving forward here?

@Mark-Simulacrum
Copy link
Member

I've filed an issue over there (rust-lang/rust-central-station#719) and will try to investigate a bit right now, it looks like it's hopefully a fairly easy fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants