[tor-bugs] #3786 [Tor Relay]: Make clients and bridges use their IPv6 address

Tor Bug Tracker & Wiki torproject-admin at torproject.org
Fri Nov 25 04:15:54 UTC 2011


#3786: Make clients and bridges use their IPv6 address
-----------------------+----------------------------------------------------
 Reporter:  ln5        |          Owner:  ln5               
     Type:  task       |         Status:  needs_review      
 Priority:  normal     |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Relay  |        Version:                    
 Keywords:             |         Parent:  #3563             
   Points:             |   Actualpoints:                    
-----------------------+----------------------------------------------------

Comment(by nickm):

 In extend_info_from_* , I think that use_pref_addr is the wrong name for
 the argument; I think instead it should be "for_direct_connect" or
 something.  That way, the caller only needs to know what they want to do
 with the extend_info; not what their needs imply about how the extend_info
 needs to be generated.

 The test in circuit_get_open_circ_or_launch looks right to me: in the case
 of a onehop circuit, the extend_info for the exit node you want should be
 the one you're using to connect to it.

 If node_get_all_orports() is at all near the critical path, we should
 think about some way to make it faster: allocating and freeing little
 objects over and over can get expensive.

 Have a look at the callers of node_get_{prim,perf}_{addr,orport} : would
 it make more sense or less sense to make these functions yield a
 tor_addr_port_t instead?

 In get_configured_bridge_by_routerinfo: if there is no bridge configured
 with the preferred address, should we fall back and check whether there is
 one configured with the primary address?  Or did I misunderstand
 something?

 Otherwise looks okay, I think.  If you think I should merge after the next
 round, please include a changes file, and let me know how tested this
 is/isn't at that point.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3786#comment:12>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list