FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-79638: Restore "Treat an unreachable robots.txt as disallow all" by serhiy-storchaka · Pull Request #152525 · python/cpython · GitHub

Repository navigation

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

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:gh-79638-restore
Jul 5, 2026
Merged

serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:gh-79638-restore

Conversation

Copy link
Copy Markdown
Member

The gh-79638 change ("Treat an unreachable robots.txt as 'disallow all'", GH-138555) was accidentally reverted by the gh-106693 / GH-149514 ob_sval revert, which removed the read() 5xx → disallow_all branch in urllib.robotparser and its NEWS entry along with the intended bytesobject.h revert.

This restores the read() behaviour and the NEWS entry. The tests were already restored separately by GH-149569.

…all"

This change was accidentally reverted by f0daba1 (pythongh-106693,
pythonGH-149514), which only intended to revert the ob_sval change.
The tests were already restored by pythonGH-149569.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added needs backport to 3.13 only security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Jun 28, 2026
serhiy-storchaka requested a review from hugovk June 28, 2026 19:33
serhiy-storchaka merged commit 70100b9 into python:main Jul 5, 2026
59 checks passed

Copy link
Copy Markdown

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

serhiy-storchaka deleted the gh-79638-restore branch July 5, 2026 11:13

bedevere-app Bot commented Jul 5, 2026

Copy link
Copy Markdown

GH-153107 is a backport of this pull request to the 3.15 branch.

bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Jul 5, 2026

bedevere-app Bot commented Jul 5, 2026

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Jul 5, 2026

bedevere-app Bot commented Jul 5, 2026

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.13 only security fixes label Jul 5, 2026
serhiy-storchaka added a commit that referenced this pull request Jul 5, 2026
… all" (GH-152525) (GH-153109)

This change was accidentally reverted by f0daba1 (gh-106693,
GH-149514), which only intended to revert the ob_sval change.
The tests were already restored by GH-149569.
(cherry picked from commit 70100b9)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added a commit that referenced this pull request Jul 5, 2026
… all" (GH-152525) (GH-153108)

This change was accidentally reverted by f0daba1 (gh-106693,
GH-149514), which only intended to revert the ob_sval change.
The tests were already restored by GH-149569.
(cherry picked from commit 70100b9)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added a commit that referenced this pull request Jul 5, 2026
… all" (GH-152525) (GH-153107)

This change was accidentally reverted by f0daba1 (gh-106693,
GH-149514), which only intended to revert the ob_sval change.
The tests were already restored by GH-149569.
(cherry picked from commit 70100b9)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL