[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #3594 [Tor Bridge]: Add support for SOCKS parameters in Bridge and {Client, Server}TransportPlugin lines



#3594: Add support for SOCKS parameters in Bridge and
{Client,Server}TransportPlugin lines
------------------------+---------------------------------------------------
 Reporter:  asn         |          Owner:                    
     Type:  defect      |         Status:  needs_review      
 Priority:  normal      |      Milestone:  Tor: 0.2.4.x-final
Component:  Tor Bridge  |        Version:                    
 Keywords:              |         Parent:  #4685             
   Points:              |   Actualpoints:                    
------------------------+---------------------------------------------------

Comment(by nickm):

 Reviewing...

   * I think the names for tor_escape_string and string_is_key_value should
 really have something to say they're only for this particular use (socks
 argument parsing/encoding).
     * It makes me sad that we are proliferating ways to escape and
 unescape stuff.
     * Can string_is_key_value ever be given escaped srings?
     * Is it really invalid to have a "K=" to indicate that K is set to the
 empty string?
     * Will the escape function ever get called with escape_char != '\\' ?
   * tor_char_is_in_set could be replaced with strchr

 Other than that, looks mergeable in 0.2.4.x.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3594#comment:9>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs