Skip to content

gh-76002: asyncio: allow pause_reading() from connection_made() - #4053

Closed
pitrou wants to merge 2 commits into
python:mainfrom
pitrou:pause_in_connection_made
Closed

gh-76002: asyncio: allow pause_reading() from connection_made()#4053
pitrou wants to merge 2 commits into
python:mainfrom
pitrou:pause_in_connection_made

Conversation

@pitrou

@pitrou pitrou commented Oct 19, 2017

Copy link
Copy Markdown
Member

@1st1

1st1 commented Oct 19, 2017

Copy link
Copy Markdown
Member

Hm. We'll need to run tests from the uvloop (has more functional tests) project with this patch.

@1st1

1st1 commented Oct 19, 2017

Copy link
Copy Markdown
Member

But I like the change. Let me think about it for a few days.

@pitrou

pitrou commented Oct 19, 2017

Copy link
Copy Markdown
Member Author

No pb!

@pitrou

pitrou commented Dec 21, 2017

Copy link
Copy Markdown
Member Author

@1st1 any update here?

self._loop.call_soon(self._loop._add_reader,
self._sock_fd, self._read_ready)
if waiter is not None:
def _call_connection_made():

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Convert nested function into instance method maybe?

@asvetlov

Copy link
Copy Markdown
Contributor

I like the PR but the came approach should be added to all transports (SSL and pipes) IMHO.

@vstinner

Copy link
Copy Markdown
Member

I removed the " needs backport to 3.6" label, the 3.6 branch no longer accept bugfixes (only security fixes are accepted): https://devguide.python.org/#status-of-python-branches

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Mar 17, 2022
@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label Jul 29, 2022
@github-actions

github-actions Bot commented Dec 8, 2022

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Dec 8, 2022
@AlexWaygood AlexWaygood changed the title bpo-31821: allow pause_reading() from connection_made() gh-76002: allow pause_reading() from connection_made() Dec 8, 2022
@AlexWaygood AlexWaygood changed the title gh-76002: allow pause_reading() from connection_made() gh-76002: asyncio: allow pause_reading() from connection_made() Dec 8, 2022
@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label Dec 9, 2022

@kumaraditya303 kumaraditya303 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There are merge conflicts.

@bedevere-bot

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Jan 10, 2023
@kumaraditya303

Copy link
Copy Markdown
Contributor

Closing as there are merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes stale Stale PR or inactive for long period of time. topic-asyncio type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.