[project @ Arch-1:robey@lag.net--2005-master-shake%paramiko--dev--1--patch-5]

split out Packetizer, fix banner detection bug, new unit test
split out a chunk of BaseTransport into a Packetizer class, which handles
the in/out packet data, ciphers, etc.  it didn't make the code any smaller
(transport.py is still close to 1500 lines, which is awful) but it did split
out a coherent chunk of functionality into a discrete unit.

in the process, fixed a bug that alain spineux pointed out: the banner
check was too forgiving and would block forever waiting for an SSH banner.
now it waits 5 seconds for the first line, and 2 seconds for each subsequent
line, before giving up.

added a unit test to test keepalive, since i wasn't sure that was still
working after pulling out Packetizer.

1 files changed, 3 insertions, 3 deletions

diff --git a/test.py b/test.py
index 2c4a28ad..a97354ee 100755
--- a/test.py
+++ b/test.py
@@ -70,6 +70,9 @@ options, args = parser.parse_args()
 if len(args) > 0:
     parser.error('unknown argument(s)')
 
+# setup logging
+paramiko.util.log_to_file('test.log')
+
 if options.use_sftp:
     if options.use_loopback_sftp:
         SFTPTest.init_loopback()
@@ -78,9 +81,6 @@ if options.use_sftp:
     if not options.use_big_file:
         SFTPTest.set_big_file_test(False)
 
-# setup logging
-paramiko.util.log_to_file('test.log')
-    
 suite = unittest.TestSuite()
 suite.addTest(unittest.makeSuite(MessageTest))
 suite.addTest(unittest.makeSuite(BufferedFileTest))