Skip to content

[3.7] bpo-32861: urllib.robotparser fix incomplete __str__ methods. (GH-5711) #6795

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

Merged
merged 4 commits into from
May 14, 2018

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 14, 2018

The urllib.robotparser's __str__ representation now includes wildcard entries and the "Crawl-delay" and "Request-rate" fields.
(cherry picked from commit bd08a0a)

Co-authored-by: Michael Lazar [email protected]

https://bugs.python.org/issue32861

…GH-5711)

The urllib.robotparser's __str__ representation now includes wildcard
entries and the "Crawl-delay" and "Request-rate" fields. Also removes extra
newlines that were being appended to the end of the string.
(cherry picked from commit bd08a0a)

Co-authored-by: Michael Lazar <[email protected]>
@miss-islington
Copy link
Contributor Author

Thanks @miss-islington for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor Author

Thanks, @serhiy-storchaka!

@miss-islington miss-islington deleted the backport-bd08a0a-3.7 branch May 14, 2018 18:14
miss-islington added a commit to miss-islington/cpython that referenced this pull request May 14, 2018
…ythonGH-5711) (pythonGH-6795)

The urllib.robotparser's __str__ representation now includes wildcard
entries and the "Crawl-delay" and "Request-rate" fields.
(cherry picked from commit bd08a0a)

Co-authored-by: Michael Lazar <[email protected]>
(cherry picked from commit c3fa1f2)

Co-authored-by: Miss Islington (bot) <[email protected]>
@miss-islington
Copy link
Contributor Author

Sorry, @miss-islington and @serhiy-storchaka, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker c3fa1f2b93fa4bf96a8aadc74ee196384cefa31e 2.7

@Mariatta
Copy link
Member

This should not have the backport labels.

serhiy-storchaka pushed a commit to serhiy-storchaka/cpython that referenced this pull request May 14, 2018
…ythonGH-5711) (pythonGH-6795)

The robotparser's __str__ representation now includes wildcard
entries.
(cherry picked from commit c3fa1f2)

Co-authored-by: Michael Lazar <[email protected]>.
@serhiy-storchaka
Copy link
Member

I have added the backport labels because the modified vwrsion should be backported to maintained releases. I modified this PR and set labels for half-automatic backporting to 3.6 and 2.7.

@miss-islington
Copy link
Contributor Author

Thanks @miss-islington for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6.
🐍🍒⛏🤖

miss-islington added a commit to miss-islington/cpython that referenced this pull request May 14, 2018
…ythonGH-5711) (pythonGH-6795)

The urllib.robotparser's __str__ representation now includes wildcard
entries and the "Crawl-delay" and "Request-rate" fields.
(cherry picked from commit bd08a0a)

Co-authored-by: Michael Lazar <[email protected]>
(cherry picked from commit c3fa1f2)

Co-authored-by: Miss Islington (bot) <[email protected]>
serhiy-storchaka pushed a commit that referenced this pull request May 14, 2018
…H-5711) (GH-6795) (GH-6818)

The urllib.robotparser's __str__ representation now includes wildcard
entries and the "Crawl-delay" and "Request-rate" fields.
(cherry picked from commit c3fa1f2)

Co-authored-by: Michael Lazar <[email protected]>
serhiy-storchaka added a commit that referenced this pull request May 14, 2018
GH-6795) (GH-6817)

The robotparser's __str__ representation now includes wildcard
entries.
(cherry picked from commit c3fa1f2)

Co-authored-by: Michael Lazar <[email protected]>.
@serhiy-storchaka serhiy-storchaka removed their assignment Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants