Skip to content

gh-75743: Restore test_timeout.testConnectTimeout() #109087

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 2 commits into from
Sep 7, 2023

Conversation

ewdurbin
Copy link
Member

@ewdurbin ewdurbin commented Sep 7, 2023

This un-skips this test now that pythontest.net implements appropriate firewall rules for it.

Alternative to #109060

This un-skips this test now that pythontest.net implements appropriate firewall
rules for it.
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

🥳

Ref: python/psf-salt@0e2a423

RETURN was relying on default REJECT, explicitly REJECT in INPUT
@vstinner
Copy link
Member

vstinner commented Sep 7, 2023

@ewdurbin: Whaaaaaaaaat? I was awaiting for that service for years, and you made it possible in 3 hours? I'm... amazed! Thanks a lot 🙏

@vstinner vstinner merged commit 1829a3c into python:main Sep 7, 2023
@vstinner vstinner added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Sep 7, 2023
@miss-islington
Copy link
Contributor

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

@miss-islington
Copy link
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 7, 2023
…09087)

This un-skips this test now that pythontest.net implements appropriate firewall
rules for it.
(cherry picked from commit 1829a3c)

Co-authored-by: Ee Durbin <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 7, 2023
…09087)

This un-skips this test now that pythontest.net implements appropriate firewall
rules for it.
(cherry picked from commit 1829a3c)

Co-authored-by: Ee Durbin <[email protected]>
@bedevere-bot
Copy link

GH-109102 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Sep 7, 2023
@vstinner
Copy link
Member

vstinner commented Sep 7, 2023

Merged, thanks again @ewdurbin!

@ewdurbin ewdurbin deleted the snakebite branch September 7, 2023 18:16
vstinner pushed a commit that referenced this pull request Sep 7, 2023
#109102)

gh-75743: Restore test_timeout.testConnectTimeout() (GH-109087)

This un-skips this test now that pythontest.net implements appropriate firewall
rules for it.
(cherry picked from commit 1829a3c)

Co-authored-by: Ee Durbin <[email protected]>
Yhg1s pushed a commit that referenced this pull request Sep 8, 2023
#109103)

gh-75743: Restore test_timeout.testConnectTimeout() (GH-109087)

This un-skips this test now that pythontest.net implements appropriate firewall
rules for it.
(cherry picked from commit 1829a3c)

Co-authored-by: Ee Durbin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants