New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure backwards compatibility for previous versions of obfsproxy (managed mode) #126

kpdyer opened this Issue Apr 1, 2014 · 1 comment


None yet
1 participant

kpdyer commented Apr 1, 2014

When running in managed mode we need to support all versions of obfsproxy in the 0.2.x chain.

Currently, we are getting

Traceback (most recent call last):
  File "/Applications/", line 88, in callWithLogger
    return callWithContext({"system": lp}, func, *args, **kw)
  File "/Applications/", line 73, in callWithContext
    return{ILogContext: newCtx}, func, *args, **kw)
  File "/Applications/", line 118, in callWithContext
    return self.currentContext().callWithContext(ctx, func, *args, **kw)
  File "/Applications/", line 81, in callWithContext
    return func(*args,**kw)
--- <exception caught here> ---
  File "/Applications/", line 151, in _doReadOrWrite
    why = getattr(selectable, method)()
  File "/Applications/", line 215, in doRead
    return self._dataReceived(data)
  File "/Applications/", line 221, in _dataReceived
    rval = self.protocol.dataReceived(data)
  File "/Applications/", line 116, in dataReceived
    self.circuit.dataReceived(self.buffer, self)
  File "/Applications/", line 154, in dataReceived
    self.transport.receivedUpstream(data, self)
exceptions.TypeError: receivedUpstream() takes exactly 2 arguments (3 given)

on obfsproxy 0.2.4. (As tested on OSX.)

@kpdyer kpdyer added this to the 0.2.11 milestone Apr 1, 2014

@kpdyer kpdyer self-assigned this Apr 1, 2014


This comment has been minimized.

Show comment
Hide comment

kpdyer Apr 1, 2014




kpdyer commented Apr 1, 2014


@kpdyer kpdyer closed this Apr 1, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment