Skip to content

Conversation

@miguendes
Copy link
Contributor

@miguendesmiguendes commented May 3, 2021

@miguendesmiguendes changed the title Fix multiprocessing Listener authkey bugbpo-43952: Fix multiprocessing Listener authkey bugMay 3, 2021
@pitrou
Copy link
Member

@aeros You may be interested in reviewing this, though it's quite a low-priority bugfix.

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity.

@github-actionsgithub-actionsbot added the stale Stale PR or inactive for long period of time. label Jun 13, 2021
@github-actionsgithub-actionsbot removed the stale Stale PR or inactive for long period of time. label Aug 7, 2022
Copy link
Member

@serhiy-storchakaserhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM

@miss-islington-app
Copy link

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

@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 27, 2024
Listener.accept() no longer hangs when authkey is an empty bytes object. (cherry picked from commit 686ec17) Co-authored-by: Miguel Brito <[email protected]>
@bedevere-app
Copy link

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

@bedevere-appbedevere-appbot removed the needs backport to 3.11 only security fixes label Feb 27, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 27, 2024
Listener.accept() no longer hangs when authkey is an empty bytes object. (cherry picked from commit 686ec17) Co-authored-by: Miguel Brito <[email protected]>
@bedevere-app
Copy link

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

@bedevere-appbedevere-appbot removed the needs backport to 3.12 only security fixes label Feb 27, 2024
serhiy-storchaka pushed a commit that referenced this pull request Feb 27, 2024
…GH-115995) Listener.accept() no longer hangs when authkey is an empty bytes object. (cherry picked from commit 686ec17) Co-authored-by: Miguel Brito <[email protected]>
serhiy-storchaka pushed a commit that referenced this pull request Feb 27, 2024
…GH-115994) Listener.accept() no longer hangs when authkey is an empty bytes object. (cherry picked from commit 686ec17) Co-authored-by: Miguel Brito <[email protected]>
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
Listener.accept() no longer hangs when authkey is an empty bytes object.
gpshead added a commit to gpshead/cpython that referenced this pull request Mar 6, 2024
Potentially introduced by python#25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ```
gpshead added a commit that referenced this pull request Mar 6, 2024
Fix some test_multiprocessing flakiness. Potentially introduced by #25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ```
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 6, 2024
…6434) Fix some test_multiprocessing flakiness. Potentially introduced by python#25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ``` (cherry picked from commit ea1803e) Co-authored-by: Gregory P. Smith <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 6, 2024
…6434) Fix some test_multiprocessing flakiness. Potentially introduced by python#25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ``` (cherry picked from commit ea1803e) Co-authored-by: Gregory P. Smith <[email protected]>
gpshead added a commit that referenced this pull request Mar 6, 2024
…GH-116440) Fix some test_multiprocessing flakiness. Potentially introduced by #25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ``` (cherry picked from commit ea1803e) Co-authored-by: Gregory P. Smith <[email protected]>
gpshead added a commit that referenced this pull request Mar 6, 2024
…GH-116441) Fix some test_multiprocessing flakiness. Potentially introduced by #25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ``` (cherry picked from commit ea1803e) Co-authored-by: Gregory P. Smith <[email protected]>
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
Listener.accept() no longer hangs when authkey is an empty bytes object.
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
Fix some test_multiprocessing flakiness. Potentially introduced by python#25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ```
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
Listener.accept() no longer hangs when authkey is an empty bytes object.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
Fix some test_multiprocessing flakiness. Potentially introduced by python#25845 not joining that thread likely leads to recently observed "environment changed" logically passing but overall failing tests seen on some buildbots similar to: ``` 1 test altered the execution environment (env changed): test.test_multiprocessing_fork.test_processes 2 re-run tests: test.test_multiprocessing_fork.test_processes test.test_multiprocessing_forkserver.test_processes ```
Sign up for freeto 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.

7 participants

@miguendes@pitrou@serhiy-storchaka@vstinner@the-knights-who-say-ni@ezio-melotti@bedevere-bot