Skip to content

bpo-43952: Fix multiprocessing Listener authkey bug - #25845

Merged
serhiy-storchaka merged 3 commits into
python:mainfrom
miguendes:bpo-43952
Feb 27, 2024
Merged

bpo-43952: Fix multiprocessing Listener authkey bug#25845
serhiy-storchaka merged 3 commits into
python:mainfrom
miguendes:bpo-43952

Conversation

@miguendes

@miguendesmiguendes commented May 3, 2021

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

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

@github-actions

Copy link
Copy Markdown

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

@serhiy-storchakaserhiy-storchaka left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown

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
Copy Markdown

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 <5544985+miguendes@users.noreply.github.com>
@bedevere-app

Copy link
Copy Markdown

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 <5544985+miguendes@users.noreply.github.com>
@bedevere-app

Copy link
Copy Markdown

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 <5544985+miguendes@users.noreply.github.com>
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 <5544985+miguendes@users.noreply.github.com>
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 <greg@krypto.org>
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 <greg@krypto.org>
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 <greg@krypto.org>
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 <greg@krypto.org>
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