[tor-bugs] #10342 [Obfsproxy]: make Circuit param a class attribute of Transport, instead of a per-method param

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Dec 11 01:53:05 UTC 2013


#10342: make Circuit param a class attribute of Transport, instead of a per-method
param
---------------------------+--------------------------
     Reporter:  infinity0  |      Owner:  infinity0
         Type:  defect     |     Status:  needs_review
     Priority:  normal     |  Milestone:
    Component:  Obfsproxy  |    Version:
   Resolution:             |   Keywords:
Actual Points:             |  Parent ID:
       Points:             |
---------------------------+--------------------------

Comment (by asn):

 Patch looks plausible. I'll need to do another pass before merging.

 I'm not sure if I want to rename `handshake()` to `circuitConnected()`
 right now. I think I would prefer to do this after bananaphone and
 scramblesuit get merged. David and Philipp, what do you think? If we do
 this change, you will also need to change your codebase to reflect that.

 In any case the `handshake()`->`circuitconnected()` rename should be a
 commit of its own. And we should probably also mention it in the
 `ChangeLog`.

 Thanks for the code :)

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


More information about the tor-bugs mailing list