Opened 6 years ago

Closed 5 years ago

Last modified 3 months ago

#5116 enhancement closed fixed (fixed)

twisted.internet.iocpreactor.iocpsupport.fillinetaddr should take an address family parameter and support AF_INET6

Reported by: PenguinOfDoom Owned by:
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/iocp-connect-ipv6-5116
branch-diff, diff-cov, branch-cov, buildbot
Author: pahan


iocpreactor's ConnectEx wrapper does not currently support AF_INET6 sockets and addresses

Change History (5)

comment:1 Changed 6 years ago by pahan

  • Author set to pahan
  • Branch set to branches/iocp-connect-ipv6-5116

(In [31910]) Branching to 'iocp-connect-ipv6-5116'

comment:2 Changed 5 years ago by glyph

  • Owner changed from PenguinOfDoom to glyph
  • Status changed from new to assigned

This is fixed in the same branch as #5085.

comment:3 Changed 5 years ago by glyph

  • Keywords review added
  • Owner glyph deleted
  • Status changed from assigned to new

Putting this up for review because the reviewer of #5085 definitely deserves some bonus points. (And, if they are so inclined, they may review this portion separately.)

comment:4 Changed 5 years ago by glyph

  • Resolution set to fixed
  • Status changed from new to closed

(In [34020]) Merge ipv6-connectTCP-5085-2.

Author: acapnotic, glyph, chjurk, ragzilla

Reviewer: exarkun

Allow Twisted to connect to IPv6 addresses, using all currently supported reactors.

IPv6 clients are supported by passing an IPv6 address literal to connectTCP. Hostname resolution is the responsibility of the caller.

This change also significantly improves the internal maintenance documentation coverage for twisted.internet.tcp and refactors more client code to be shared between the IOCP reactor and all the UNIX reactors.

Fixes: #5085 Fixes: #5116

comment:5 Changed 3 months ago by hawkowl

  • Keywords review removed

[mass edit] Removing review from closed tickets.

Note: See TracTickets for help on using tickets.