-
-
Notifications
You must be signed in to change notification settings - Fork 33.7k
bpo-40121: Fix exception type in test #19267
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
Conversation
|
Anyone can feel free to merge this once CI passes. |
| # Don't care if this fails, we just want the audit message | ||
| sock.bind(('127.0.0.1', 8080)) | ||
| except error: | ||
| except Exception: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may be more specific and only catch OSError.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's only a test, and I'd rather not risk breaking buildbots over it.
|
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8. |
|
GH-19275 is a backport of this pull request to the 3.8 branch. |
(cherry picked from commit 3ef4a7e) Co-authored-by: Steve Dower <[email protected]>
|
|
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8. |
(cherry picked from commit 3ef4a7e) Co-authored-by: Steve Dower <[email protected]>
|
GH-19276 is a backport of this pull request to the 3.8 branch. |
(cherry picked from commit 3ef4a7e) Co-authored-by: Steve Dower <[email protected]>
https://bugs.python.org/issue40121