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

Re: [tor-bugs] #5083 [Obfsproxy]: Implement heartbeat message in obfsproxy



#5083: Implement heartbeat message in obfsproxy
-------------------------+--------------------------------------------------
 Reporter:  karsten      |          Owner:  asn         
     Type:  enhancement  |         Status:  needs_review
 Priority:  normal       |      Milestone:              
Component:  Obfsproxy    |        Version:              
 Keywords:               |         Parent:              
   Points:               |   Actualpoints:              
-------------------------+--------------------------------------------------

Comment(by karsten):

 Replying to [comment:11 nickm]:
 > Reviewing the whole series as one diff...
 >
 > Why strmap_set_lc, and not strmap_set?

 I was thinking of two IPv6 addresses with uppercase and lowercase hex
 characters that should be considered equal.  If we're sure that doesn't
 happen, strmap_set should be fine.

 > Should "connections" be static?

 Yes!  Fixed in the updated heartbeat_uniqueips branch.

 I also found another problem with strmap_t not liking NULL values, even
 though we don't care about values.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/5083#comment:12>
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