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.

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

feat(client): make session safe in multiprocesses environment #382

Merged
merged 1 commit into from
Apr 15, 2021

Conversation

AChenQ
Copy link
Collaborator

@AChenQ AChenQ commented Apr 14, 2021

SSL may raise error when reusing the same session for multiprocesses

related issue:

psf/requests#4323

@AChenQ AChenQ requested review from linjiX and rexzheng324-c April 14, 2021 06:47
@@ -12,6 +12,7 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

is a feat or a fix

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In my opinion, it is a new feature to support multiprocesses security.

@AChenQ AChenQ force-pushed the T14418_multiprocess_safe branch from 42ec444 to 08a2fb4 Compare April 14, 2021 07:55
@AChenQ AChenQ force-pushed the T14418_multiprocess_safe branch from 08a2fb4 to 1919e22 Compare April 14, 2021 08:24
AChenQ pushed a commit to AChenQ/tensorbay-python-sdk that referenced this pull request Apr 15, 2021
SSL may raise error when reusing the same session for multiprocesses

related issue:

psf/requests#4323

PR Closed: Graviti-AI#382
@AChenQ AChenQ force-pushed the T14418_multiprocess_safe branch from 1919e22 to 0c17b2d Compare April 15, 2021 07:33
SSL may raise error when reusing the same session for multiprocesses

related issue:

psf/requests#4323

PR Closed: Graviti-AI#382
@AChenQ AChenQ force-pushed the T14418_multiprocess_safe branch from c8d840c to 07bc589 Compare April 15, 2021 07:37
@AChenQ AChenQ merged commit 754d74e into Graviti-AI:main Apr 15, 2021
@AChenQ AChenQ deleted the T14418_multiprocess_safe branch April 15, 2021 07:46
AChenQ pushed a commit to AChenQ/tensorbay-python-sdk that referenced this pull request Apr 16, 2021
SSL may raise error when reusing the same session for multiprocesses

related issue:

psf/requests#4323

PR Closed: Graviti-AI#382
linjiX pushed a commit that referenced this pull request Apr 19, 2021
SSL may raise error when reusing the same session for multiprocesses

related issue:

psf/requests#4323

PR Closed: #382
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants