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
gh-79200: support IPAddress in asyncio.start_server() #17434
Conversation
Hello, and thanks for your contribution! I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA). Recognized GitHub usernameWe couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames: This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue. You can check yourself to see if the CLA has been received. Thanks again for the contribution, we look forward to reviewing it! |
Notes: I did put an issue # in the title. |
The new interface (DeviceListener) has two methods: * connection_lost * connection_closed In case of a problem, e.g. server closes connection, connection_lost is called. In case of a controlled shutdown (e.g. user calls close), connection_closed is called instead.
It usually takes around 24-48 hours before the CLI status is updated on GitHub, I'd recommend periodically using the If it's been more than 48 hours since you submitted the CLA, you can reach out to the PSF at either: |
Hi @vegerot, thanks for fixing this! Can you please:
- change the title of the pull request to "bpo-35019: support IPAddress in asyncio.start_server()"
- Add a test for this change
- Add a blurb, you can add one by using https://blurb-it.herokuapp.com/
- Add a
.. versionchanged::
note inDoc/library/asyncio-stream.rst
to document this change - Add your name to
Misc/ACKS
?
Thanks!
@ericvsmith @remilapeyre thanks for reviewing my PR so in depth |
My bad. I will look over that now. I messed up my fast-forward and ended up with thousands of commits in this PR |
I've certainly done that before (when I first started contributing). My recommendation is to use
See the git bootcamp section of the devguide for details. |
@asvetlov mentioned taking a more generalized approach in the original bpo issue:
Andrew, do you still have the same thoughts on the issue, or should we consider a more simplified solution? I would certainly prefer something like an IPLike approach so we don't have to make changes in future and current asyncio APIs that accept IP addresses, but if that's going to be significantly far away, it may be worthwhile to consider the convenience benefits for just the high-level parts of asyncio (such as |
@aeros |
Any update on this? |
Closing as won't fix as discussed on issue. |
@kumaraditya303 Where is the issue? |
|
Add support for new Python3 IPv4Address inet type. This is tested and forwards and backwards compatible with different host types.