Socks V5 functionality

[jadestorm]

One of my PyAIM/PyICQ users submitted this code and it works like a charm. 
Thought it might be nice to include in the main tree.  I only see Socks V4
support as it stands.  There are two files here, I'll attach the other one in a
second.  I imagine there's no good reason why the second (sockserror.py) file
couldn't be merged into socks5.py, but hey.  Daniel Chandran
(daniel.chandran@gmail.com) submitted this code.  I offer it to Twisted proper
with his blessing.  =)

[jadestorm]

Here's a quick example of using it; you'll note that there is a check for
whether socks is configured so that it's obvious what the difference between a
normal connection and a socks 5 based connection is:

        if self.socksProxyServer and self.socksProxyPort:
            c = protocol.ProxyClientCreator(reactor, serviceClasses[service],
self, cookie, d)
            c.connectSocks5Proxy(ip, 5190, self.socksProxyServer,
int(self.socksProxyPort), "BOSCONN").addCallback(addService)
        else:
            c = protocol.ClientCreator(reactor, serviceClasses[service], self,
cookie, d)       
            c.connectTCP(ip, 5190).addCallback(addService)

[jknight]

This looks like a nice feature but it needs tests.

[khorn]

FYI, This looks like the same code from this email message:  http://twistedmatrix.com/pipermail/twisted-python/2003-February/003057.html except split into two files.

Assuming it is, the original author was Dmitry Litovchenko.

[glyph]

This would need test coverage, and some kind of OK from Mr. Litovchenko if it were to be included.

[khorn]

There is some server-side SOCKSv5 code here:  http://code.google.com/p/proxy65/source/browse/trunk/proxy65/socks5.py

It's MIT-licensed. Might be a good match with the client code?

[khorn]

Replying to glyph:

This would need test coverage, and some kind of OK from Mr. Litovchenko if it were to be included.

Seeing as how Mr. Litovchenko sent it to the Twisted list with the apparent intention of contributing it, I think it's probably OK to include it.

Still does need tests though.

[khorn]

I managed to track down Mr. Litovchenko. Here's the text of my email:

Dmitry,

A number of years ago, you submitted some SOCKS5 client code to the Twisted python mailing list.  Here is the mail, in case you don't recall:
http://twistedmatrix.com/pipermail/twisted-python/2003-February/003057.html

Some time later, a ticket was opened on the Twisted Trac instance which contained code very similar (possibly identical) to yours.  Here is that ticket:
http://twistedmatrix.com/trac/ticket/1330

In the interests of moving this ticket along, I have a few questions:

- Can you confirm that this code is in fact yours?
- Can you further confirm that it was your intention to have this code included in Twisted?
- Were you ever able to create any unit tests for this code?

Thanks,

Kevin Horn

and here's the text of his response:

1. Right, this looks like my old code just was split into 2 files + possibly some edits were made to it.
2. If you have intention to put it into Twisted codebase, go ahead, i'll be happy to know it finally made it in :)
3. No i think i didn't continue work on the module or unit tests, after it started doing that i needed from it.

So now this definitely just needs tests (and possibly to be updated to the current version of Twisted).