[Pidgin] #11290: add bonjour ipv6 support

Pidgin trac at pidgin.im
Fri Feb 19 07:14:18 EST 2010


#11290: add bonjour ipv6 support
------------------------------------+---------------------------------------
 Reporter:  tux                     |        Owner:  datallah
     Type:  patch                   |       Status:  new     
Milestone:  Patches Needing Review  |    Component:  Bonjour 
  Version:  2.6.5                   |   Resolution:          
 Keywords:  ipv6 bonjour            |  
------------------------------------+---------------------------------------

Comment(by tux):

 Replying to [comment:5 datallah]:
 > I just added a new version of the patch with the following changes:
 >  * resolving a buddy only occurs for the protocol that we received a
 presence notification on
 >  * IPv6 buddy IPs are preferred to IPv4 IPs
 >   * if there are multiples of either protocol, the one (within the
 protocol) that actually gets used continues to be arbitrary
 >  * Update the file transfer code to support IPv6
 >   * I don't believe that the core purple_network_listen code is doing
 the right thing yet, but if it does, then bonjour file transfers *should*
 work.
 Yes looks pretty good, I tested it for a couple of hours and works as
 expected, the connection has always been establised via ipv6 in our setup.
 And yes, you're right, purple_network_listen() does not do the right
 thing... it just opens an ipv4 socket listener, not an ipv6 one.
 Another thing I noticed is, that after a bonjour: "_resolve_callback -
 Failure: Timeout reached" + "bonjour: _browser_callback - Remove service"
 the buddy is being removed from the buddy list, yes, but in the still open
 chat window he's just shown as away and not offline.
 Despite from those things, the last patch seems to work pretty well and
 can can be applied to leave the other two things up to a new ticket.

 Cheers, T_X

-- 
Ticket URL: <http://developer.pidgin.im/ticket/11290#comment:7>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list