Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Bug]: multi_process_shared.py doesn't correctly handle cross process authentication #26167

Closed
1 of 15 tasks
damccorm opened this issue Apr 7, 2023 · 1 comment · Fixed by #26172
Closed
1 of 15 tasks

[Bug]: multi_process_shared.py doesn't correctly handle cross process authentication #26167

damccorm opened this issue Apr 7, 2023 · 1 comment · Fixed by #26172
Assignees
@damccorm
Labels
bug done & done Issue has been reviewed after it was closed for verification, followups, etc. P2 python
Milestone
2.48.0 Release

Comments

@damccorm
Copy link
Contributor

damccorm commented Apr 7, 2023

What happened?

Right now when trying to access objects across processes, multi_process_shared.py doesn't set an auth key. When that happens, the manager still creates an auth key for that process based on the current process's authKey - https://github.com/python/cpython/blob/4dc339b4d69195448207e1faecc3e258700daf33/Lib/multiprocessing/managers.py#L503

This causes cross-process authentication to fail and means that multi_process_shared.py doesn't correctly work.

Issue Priority

Priority: 2 (default / most bugs should be filed as P2)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner
@damccorm damccorm self-assigned this Apr 7, 2023
@damccorm
Copy link
Contributor Author

damccorm commented Apr 7, 2023

Note: this was masked by #26168

@damccorm damccorm mentioned this issue Apr 7, 2023
Merged
3 tasks
@github-actions github-actions bot added this to the 2.48.0 Release milestone Apr 17, 2023
@tvalentyn tvalentyn added the done & done Issue has been reviewed after it was closed for verification, followups, etc. label Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@damccorm damccorm

Labels
bug done & done Issue has been reviewed after it was closed for verification, followups, etc. P2 python
Projects
None yet
Milestone
2.48.0 Release
Development

Successfully merging a pull request may close this issue.

Set an auth key in multi_process_shared.py damccorm/beam
2 participants
@tvalentyn @damccorm