On Thu, Oct 1, 2009 at 10:21 PM, Benjamin Rutt <[email protected]> wrote:

> (A/B) it looks like the SSHChannel implementation of getPeer() is faulty.
> If I change my two-line implementation of getPeer() inside
> SSHSessionProcessProtocol to simply 'return self.session.getPeer()', I get:
>
>   File
> "/home/ruttbe/dev/python/twisted-8.2.0-inst/lib/python/twisted/conch/ssh/session.py",
> line 216, in getPeer
>     return self.session.getPeer()
>   File
> "/home/ruttbe/dev/python/twisted-8.2.0-inst/lib/python/twisted/conch/ssh/channel.py",
> line 261, in getPeer
>     return('SSH', )+self.conn.transport.getPeer()
> AttributeError: SSHServerTransport instance has no attribute 'getPeer'
>

OK, I give up :).  Looks like that is pretty clearly a bug, though.


> (C) regarding adding a test, which file do you think would be suitable
> here?  something in the twisted/test dir?
>

I am not sure which exact file, but something of the form
twisted/conch/test/test_<something>.py

Paul Swartz (z3p) should really weigh in on this.  You can find him on IRC
most days, although hopefully he's reading the mailing list :).
_______________________________________________
Twisted-Python mailing list
[email protected]
http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python

Reply via email to