Excellent! I was thinking of making this change but lately I haven't had much time. Merging that patch specified in the 1st ticket comment? That looks good.
I'd be happy to update the bananaphone transport to use the new api!
Cheers,
David
On Wed, Dec 11, 2013 at 10:33 PM, George Kadianakis desnacked@riseup.netwrote:
Hello Philipp and David,
I'm sending you an email because you are the people who are currently developing obfsproxy modules.
We are thinking of slightly changing the API of obfsproxy, so that it becomes more friendly to transport authors (or at least we hope so).
Specifically, we are thinking of having the circuit be an element of the transport class, instead of being passed as an argument in the receivedDownstream and receivedUpstream methods. I think David and Leif were also thinking of this change. Furthermore, we are also thinking of renaming the 'handshake' method to 'circuitConnected'. (More details can be found in #10342.)
Do you agree with the changes? If yes, I will merge Ximin's patch for #10342, and you will have to update your transport codebase appropriately. Please let me know.
Thanks!