Skip to content

/ cpython

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

bpo-38856: Fix StreamWriter.wait_closed() ConnectionResetError for ProactorEventLoop #18199

Open
wants to merge 2 commits into
base: master
from
Open

bpo-38856: Fix StreamWriter.wait_closed() ConnectionResetError for ProactorEventLoop #18199

wants to merge 2 commits into from
+9 −6

Conversation

@aeros
Copy link
Member

aeros commented Jan 27, 2020 •
edited by bedevere-bot

https://bugs.python.org/issue38856
@aeros aeros changed the title Fix IocpProactor.recv() error bpo38856: Fix IocpProactor.recv() error Jan 27, 2020
@aeros aeros changed the title bpo38856: Fix IocpProactor.recv() error bpo-38856: Fix IocpProactor.recv() error Jan 27, 2020
@aeros aeros added the OS-windows label Jan 27, 2020
@aeros aeros changed the title bpo-38856: Fix IocpProactor.recv() error bpo-38856: Fix StreamWriter.wait_closed() ConnectionResetError for ProactorEventLoop Jan 27, 2020
@aeros aeros added the DO-NOT-MERGE label Jan 27, 2020
@aeros
Fix StreamWriter.wait_closed() ConnectionResetError for ProactorEvent…
Loading status checks…
ddfa933 
…Loop
@aeros aeros force-pushed the aeros:bpo38856-fix-IocpProactor.recv-error branch from 6a66228 to ddfa933 Jan 27, 2020
@aeros
Remove whitespace change
Loading status checks…
de24269
@aeros aeros requested a review from asvetlov Jan 27, 2020
@aeros aeros marked this pull request as ready for review Jan 27, 2020
@aeros aeros requested a review from 1st1 as a code owner Jan 27, 2020
@aeros

This comment has been minimized.

Sign in to view
Copy link
Member Author

aeros commented Jan 27, 2020

I'll leave the DO-NOT-MERGE label until someone experienced with the win32 API and its interaction with asyncio's streams API (such as @asvetlov) gets a chance to review the PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@1st1 1st1

@asvetlov asvetlov

Assignees
No one assigned
Labels
CLA signed DO-NOT-MERGE OS-windows awaiting review expert-asyncio skip news
Projects
None yet
Milestone
No milestone
3 participants
@aeros @the-knights-who-say-ni @bedevere-bot
You can’t perform that action at this time.