Skip to content

gh-79638: Treat an unreachable robots.txt as "disallow all"#138555

Merged
serhiy-storchaka merged 5 commits intopython:mainfrom
serhiy-storchaka:robotparser-unreachable
May 7, 2026
Merged

gh-79638: Treat an unreachable robots.txt as "disallow all"#138555
serhiy-storchaka merged 5 commits intopython:mainfrom
serhiy-storchaka:robotparser-unreachable

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka commented Sep 5, 2025

Disallow all access in urllib.robotparser if the robots.txt file is unreachable due to server or network errors.

Disallow all access in urllib.robotparser if the robots.txt file is
unreachable due to server or network errors.
@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 29, 2026
@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) May 4, 2026 19:45
@serhiy-storchaka serhiy-storchaka disabled auto-merge May 4, 2026 19:49
@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) May 5, 2026 09:09
@serhiy-storchaka serhiy-storchaka merged commit 310fe88 into python:main May 7, 2026
49 of 52 checks passed
@miss-islington-app
Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 7, 2026

GH-149512 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label May 7, 2026
@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 7, 2026

GH-149513 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label May 7, 2026
hugovk added a commit to hugovk/cpython that referenced this pull request May 7, 2026
@hugovk
Copy link
Copy Markdown
Member

hugovk commented May 7, 2026

This had been set to automerge but was failing CI, so wasn't meant to merge right now.

I'd temporarily removed branch protection to be able to merge #149500 which was needed to fix configure, which meant this PR also merged.

I've reverted it in #149514 to get main back to green. Sorry about this.

@serhiy-storchaka Would you like me to re-open a fresh version of this PR once that is sorted?

@serhiy-storchaka
Copy link
Copy Markdown
Member Author

I'll recreate it tomorrow.

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

Labels

stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants