Skip to content

gh-127734: improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__ #127735

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 1 commit into from
Dec 8, 2024

Conversation

tungol
Copy link
Contributor

@tungol tungol commented Dec 8, 2024

A small MR to improve signature introspection of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__. The parent class takes no arguments, so the pass-through serves no purpose. Currently, typeshed needs to have urllib.request.HTTPPasswordMgrWithPriorAuth.__init__ on its allowlist for stubtest errors as a result of the poor introspection.

Copy link
Member

@orsenthil orsenthil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@orsenthil orsenthil added skip news 3.12 only security fixes labels Dec 8, 2024
@orsenthil orsenthil merged commit a03efb5 into python:main Dec 8, 2024
46 checks passed
@orsenthil orsenthil added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Dec 8, 2024
@miss-islington-app
Copy link

Thanks @tungol for the PR, and @orsenthil for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @tungol for the PR, and @orsenthil for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@orsenthil orsenthil removed the 3.12 only security fixes label Dec 8, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 8, 2024
…WithPriorAuth.__init__` (pythonGH-127735)

improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__
(cherry picked from commit a03efb5)

Co-authored-by: Stephen Morton <git@tungol.org>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 8, 2024
…WithPriorAuth.__init__` (pythonGH-127735)

improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__
(cherry picked from commit a03efb5)

Co-authored-by: Stephen Morton <git@tungol.org>
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2024

GH-127744 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 Dec 8, 2024
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2024

GH-127745 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Dec 8, 2024
orsenthil pushed a commit that referenced this pull request Dec 8, 2024
…rWithPriorAuth.__init__` (GH-127735) (#127744)

gh-127734: improve signature of `urllib.request.HTTPPasswordMgrWithPriorAuth.__init__` (GH-127735)

improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__
(cherry picked from commit a03efb5)

Co-authored-by: Stephen Morton <git@tungol.org>
orsenthil pushed a commit that referenced this pull request Dec 8, 2024
…rWithPriorAuth.__init__` (GH-127735) (#127745)

gh-127734: improve signature of `urllib.request.HTTPPasswordMgrWithPriorAuth.__init__` (GH-127735)

improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__
(cherry picked from commit a03efb5)

Co-authored-by: Stephen Morton <git@tungol.org>
@tungol tungol deleted the urllib branch December 8, 2024 20:18
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…WithPriorAuth.__init__` (python#127735)

improve signature of urllib.request.HTTPPasswordMgrWithPriorAuth.__init__
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants