Skip to content

support various address families in uasyncio open_connection - #7563

Closed
oclyke wants to merge 1 commit into
micropython:masterfrom
oclyke:uasyncio-ipv6
Closed

oclyke wants to merge 1 commit into
micropython:masterfrom
oclyke:uasyncio-ipv6

Conversation

@oclyke

@oclyke oclyke commented Jul 22, 2021 •

Copy link
Copy Markdown
Contributor

rudimentary support for various address families
only tested on esp32 and unix ports

encountered trouble using open_connection in my codebase when adding support for the unix port. after light investigation i found that the unix port was returning ipv6 addresses for socket.getaddrinfo() in some cases (notably www.google.com) which would cause open_connection to raise OSError 47 (MP_EAFNOSUPPORT).

using the family tuple element returned by socket.getaddrinfo seems like it ought to be a portable way to adapt to the proper address family when creating a socket.

i've tried this out successfully on unix and esp32 ports, but clearly a more rigorous examination is needed. furthermore this is most likely just a fraction of the work needed to be done to claim that uasyncio truly supports ipv6

related issues:
#3683

@oclyke
oclyke marked this pull request as ready for review July 22, 2021 23:48
@dpgeorge dpgeorge added the extmod Relates to extmod/ directory in source label Jul 23, 2021
Comment thread extmod/uasyncio/stream.py Outdated
…en_connection

rudimentary support for various address families
only tested on esp32 and unix ports

Signed-off-by: oclyke <[email protected]>
@dpgeorge

Copy link
Copy Markdown
Member

Thanks! Merged in e29259d

@dpgeorge dpgeorge closed this Jul 31, 2021
@oclyke
oclyke deleted the uasyncio-ipv6 branch November 10, 2021 02:02
tannewt added a commit to tannewt/circuitpython that referenced this pull request Feb 15, 2023
…set-on-raise

Deinit the reset pin when displayio.I2CDisplay fails
@darconeous darconeous mentioned this pull request Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extmod Relates to extmod/ directory in source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants