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

Templatize robots.txt #478

Merged
merged 2 commits into from
Jan 29, 2023
Merged

Conversation

tabletcorry
Copy link
Contributor

The existing static robots.txt did not allow any method of adding additional disallowed agents.

Since a substantial majority of web traffic to my test instance is a few SEO crawlers, and this is likely to be true for others as well, add a method by which well-behaving robots can be blocked.

The existing static robots.txt did not allow any method of adding additional disallowed agents.

Since a substantial majority of web traffic to my test instance is a few SEO crawlers, and this is likely to be true for others as well, add a method by which well-behaving robots can be blocked.
@tabletcorry
Copy link
Contributor Author

I don't think the pre-commit error is due to my change 🤔

@shuuji3
Copy link
Contributor

shuuji3 commented Jan 29, 2023

It looks like the pre-commit error is due to isort: PyCQA/isort#2077

It was already fixed in v5.12.0, so I'll create a PR to upgrade the version.

@andrewgodwin andrewgodwin merged commit 93c0af9 into jointakahe:main Jan 29, 2023
@andrewgodwin
Copy link
Member

I was thinking about doing this anyway, so thanks!

@tabletcorry tabletcorry deleted the template_robots branch January 29, 2023 19:28
@tabletcorry
Copy link
Contributor Author

The two that were annoying me the most for constant crawling are TAKAHE_ROBOTS_TXT_DISALLOWED_USER_AGENTS='["AhrefsBot","SemrushBot"]' in case anyone wants to copy a few easy ones.

ephes pushed a commit to ephes/takahe that referenced this pull request Feb 7, 2023
shuuji3 pushed a commit to shuuji3/takahe that referenced this pull request Feb 22, 2023
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.

3 participants