[Python-checkins] r74376 - in python/branches/release31-maint: Lib/telnetlib.py Lib/test/test_telnetlib.py Misc/ACKS Misc/NEWS

Thu Aug 13 10:39:33 CEST 2009

Author: georg.brandl
Date: Thu Aug 13 10:39:33 2009
New Revision: 74376
Log:
Merged revisions 74217,74224 via svnmerge from 
svn+ssh://svn.python.org/python/branches/py3k
........
 r74217 | jack.diederich | 2009年07月27日 00:23:04 +0200 (Mo, 27 Jul 2009) | 1 line
 
 - fix issue #6106, Telnet.process_rawq default handling of WILL/WONT/DO/DONT
........
 r74224 | jack.diederich | 2009年07月27日 11:03:14 +0200 (Mo, 27 Jul 2009) | 1 line
 
 - belated ACK for issue #6106
........
Modified:
 python/branches/release31-maint/ (props changed)
 python/branches/release31-maint/Lib/telnetlib.py
 python/branches/release31-maint/Lib/test/test_telnetlib.py
 python/branches/release31-maint/Misc/ACKS
 python/branches/release31-maint/Misc/NEWS
Modified: python/branches/release31-maint/Lib/telnetlib.py
==============================================================================

--- python/branches/release31-maint/Lib/telnetlib.py	(original)
+++ python/branches/release31-maint/Lib/telnetlib.py	Thu Aug 13 10:39:33 2009
@@ -459,7 +459,7 @@
 # unless we did a WILL/DO before.
 self.msg('IAC %d not recognized' % ord(c))
 elif len(self.iacseq) == 2:
- cmd = self.iacseq[1]
+ cmd = self.iacseq[1:2]
 self.iacseq = b''
 opt = c
 if cmd in (DO, DONT):
Modified: python/branches/release31-maint/Lib/test/test_telnetlib.py
==============================================================================
--- python/branches/release31-maint/Lib/test/test_telnetlib.py	(original)
+++ python/branches/release31-maint/Lib/test/test_telnetlib.py	Thu Aug 13 10:39:33 2009
@@ -3,6 +3,8 @@
 import telnetlib
 import time
 import queue
+import sys
+import io
 
 from unittest import TestCase
 from test import support
@@ -304,6 +306,20 @@
 self.sb_seen += sb_data
 
 tl = telnetlib
+
+class TelnetDebuglevel(tl.Telnet):
+ ''' Telnet-alike that captures messages written to stdout when
+ debuglevel > 0
+ '''
+ _messages = ''
+ def msg(self, msg, *args):
+ orig_stdout = sys.stdout
+ sys.stdout = fake_stdout = io.StringIO()
+ tl.Telnet.msg(self, msg, *args)
+ self._messages += fake_stdout.getvalue()
+ sys.stdout = orig_stdout
+ return
+
 class OptionTests(TestCase):
 setUp = _read_setUp
 tearDown = _read_tearDown
@@ -363,6 +379,36 @@
 self.assertEqual(b'', telnet.read_sb_data())
 nego.sb_getter = None # break the nego => telnet cycle
 
+ def _test_debuglevel(self, data, expected_msg):
+ """ helper for testing debuglevel messages """
+ self.setUp()
+ self.dataq.put(data)
+ telnet = TelnetDebuglevel(HOST, self.port)
+ telnet.set_debuglevel(1)
+ self.dataq.join()
+ txt = telnet.read_all()
+ self.assertTrue(expected_msg in telnet._messages,
+ msg=(telnet._messages, expected_msg))
+ self.tearDown()
+
+ def test_debuglevel(self):
+ # test all the various places that self.msg(...) is called
+ given_a_expect_b = [
+ # Telnet.fill_rawq
+ (b'a', ": recv b''\n"),
+ # Telnet.process_rawq
+ (tl.IAC + bytes([88]), ": IAC 88 not recognized\n"),
+ (tl.IAC + tl.DO + bytes([1]), ": IAC DO 1\n"),
+ (tl.IAC + tl.DONT + bytes([1]), ": IAC DONT 1\n"),
+ (tl.IAC + tl.WILL + bytes([1]), ": IAC WILL 1\n"),
+ (tl.IAC + tl.WONT + bytes([1]), ": IAC WONT 1\n"),
+ # Telnet.write
+ # XXX, untested
+ ]
+ for a, b in given_a_expect_b:
+ self._test_debuglevel([a, EOF_sigil], b)
+ return
+
 def test_main(verbose=None):
 support.run_unittest(GeneralTests, ReadTests, OptionTests)
 
Modified: python/branches/release31-maint/Misc/ACKS
==============================================================================
--- python/branches/release31-maint/Misc/ACKS	(original)
+++ python/branches/release31-maint/Misc/ACKS	Thu Aug 13 10:39:33 2009
@@ -708,6 +708,7 @@
 Ken Stox
 Dan Stromberg
 Daniel Stutzbach
+Pal Subbiah
 Nathan Sullivan
 Mark Summerfield
 Hisao Suzuki
@@ -800,6 +801,7 @@
 Lars Wirzenius
 Chris Withers
 Stefan Witzel
+Irek Wlizlo
 David Wolever
 Klaus-Juergen Wolf
 Dan Wolfe
Modified: python/branches/release31-maint/Misc/NEWS
==============================================================================
--- python/branches/release31-maint/Misc/NEWS	(original)
+++ python/branches/release31-maint/Misc/NEWS	Thu Aug 13 10:39:33 2009
@@ -45,6 +45,9 @@
 Library
 -------
 
+- Issue #6106: telnetlib.Telnet.process_rawq doesn't handle default WILL/WONT
+ DO/DONT correctly.
+
 - Issue #6126: Fixed pdb command-line usage.
 
 - Issue #6629: Fix a data corruption issue in the new I/O library, which could
@@ -62,32 +65,32 @@
 - Issue #2715: Remove remnants of Carbon.File from binhex module.
 
 - Issue #6595: The Decimal constructor now allows arbitrary Unicode
- decimal digits in input, as recommended by the standard. Previously
- it was restricted to accepting [0-9].
+decimal digits in input, as recommended by the standard. Previously
+it was restricted to accepting [0-9].
 
 - Issues #5155, #5313, #5331: multiprocessing.Process._bootstrap was
- unconditionally calling "os.close(sys.stdin.fileno())" resulting in file
- descriptor errors
+unconditionally calling "os.close(sys.stdin.fileno())" resulting in file
+descriptor errors
 
 - Issue #1424152: Fix for http.client, urllib.request to support SSL while
- working through proxy. Original patch by Christopher Li, changes made by
- Senthil Kumaran
+working through proxy. Original patch by Christopher Li, changes made by
+Senthil Kumaran
 
 - importlib.abc.PyLoader did not inherit from importlib.abc.ResourceLoader like
- the documentation said it did even though the code in PyLoader relied on the
- abstract method required by ResourceLoader.
+the documentation said it did even though the code in PyLoader relied on the
+abstract method required by ResourceLoader.
 
 - Issue #6431: Make Fraction type return NotImplemented when it doesn't
- know how to handle a comparison without loss of precision. Also add
- correct handling of infinities and nans for comparisons with float.
+know how to handle a comparison without loss of precision. Also add
+correct handling of infinities and nans for comparisons with float.
 
 - Issue #6415: Fixed warnings.warn segfault on bad formatted string.
 
 - Issue #6358: The exit status of a command started with os.popen() was
- reported differently than it did with python 2.x.
+reported differently than it did with python 2.x.
 
 - Issue #6323: The pdb debugger did not exit when running a script with a
- syntax error.
+syntax error.
 
 - Issue #6369: Fix an RLE decompression bug in the binhex module.
 
@@ -97,16 +100,16 @@
 -----
 
 - Issue 4601: 'make install' did not set the appropriate permissions on
- directories.
+directories.
 
 - Issue 5390: Add uninstall icon independent of whether file
- extensions are installed.
+extensions are installed.
 
 Test
 ----
 
 - Fix a test in importlib.test.source.test_abc_loader that was incorrectly
- testing when a .pyc file lacked an code object bytecode.
+testing when a .pyc file lacked an code object bytecode.
 
 
 What's New in Python 3.1?
@@ -118,27 +121,27 @@
 -----------------
 
 - Issue #6334: Fix bug in range length calculation for ranges with
- large arguments.
+large arguments.
 
 - Issue #6329: Fixed iteration for memoryview objects (it was being blocked
- because it wasn't recognized as a sequence).
+because it wasn't recognized as a sequence).
 
 Library
 -------
 
 - Issue #6314: logging.basicConfig() performs extra checks on the "level"
- argument.
+argument.
 
 - Issue #6274: Fixed possible file descriptors leak in subprocess.py
 
 - Accessing io.StringIO.buffer now raises an AttributeError instead of
- io.UnsupportedOperation.
+io.UnsupportedOperation.
 
 - Issue #6271: mmap tried to close invalid file handle (-1) when anonymous.
- (On Unix)
+(On Unix)
 
 - Issue #1202: zipfile module would cause a struct.error when attempting to
- store files with a CRC32 > 2**31-1.
+store files with a CRC32 > 2**31-1.
 
 Extension Modules
 -----------------
@@ -157,50 +160,50 @@
 - Fixed SystemError triggered by "range([], 1, -1)".
 
 - Issue #5924: On Windows, a large PYTHONPATH environment variable
- (more than 255 characters) would be completely ignored.
+(more than 255 characters) would be completely ignored.
 
 - Issue #4547: When debugging a very large function, it was not always
- possible to update the lineno attribute of the current frame.
+possible to update the lineno attribute of the current frame.
 
 - Issue #5330: C functions called with keyword arguments were not reported by
- the various profiling modules (profile, cProfile). Patch by Hagen Fürstenau.
+the various profiling modules (profile, cProfile). Patch by Hagen Fürstenau.
 
 Library
 -------
 
 - Issue #6438: Fixed distutils.cygwinccompiler.get_versions : the regular
- expression string pattern was trying to match against a bytes returned by 
- Popen. Tested under win32 to build the py-postgresql project.
+expression string pattern was trying to match against a bytes returned by 
+Popen. Tested under win32 to build the py-postgresql project.
 
 - Issue #6258: Support AMD64 in bdist_msi.
 
 - Issue #6195: fixed doctest to no longer try to read 'source' data from
- binary files.
+binary files.
 
 - Issue #5262: Fixed bug in next rollover time computation in
- TimedRotatingFileHandler.
+TimedRotatingFileHandler.
 
 - Issue #6217: The C implementation of io.TextIOWrapper didn't include the
- errors property. Additionally, the errors and encoding properties of StringIO
- are always None now.
+errors property. Additionally, the errors and encoding properties of StringIO
+are always None now.
 
 - Issue #6137: The pickle module now translates module names when loading
- or dumping pickles with a 2.x-compatible protocol, in order to make data
- sharing and migration easier. This behaviour can be disabled using the
- new `fix_imports` optional argument.
+or dumping pickles with a 2.x-compatible protocol, in order to make data
+sharing and migration easier. This behaviour can be disabled using the
+new `fix_imports` optional argument.
 
 - Removed the ipaddr module.
 
 - Issue #3613: base64.{encode,decode}string are now called
- base64.{encode,decode}bytes which reflects what type they accept and return.
- The old names are still there as deprecated aliases.
+base64.{encode,decode}bytes which reflects what type they accept and return.
+The old names are still there as deprecated aliases.
 
 - Issue #5767: Remove sgmlop support from xmlrpc.client.
 
 - Issue #6150: Fix test_unicode on wide-unicode builds.
 
 - Issue #6149: Fix initialization of WeakValueDictionary objects from non-empty
- parameters.
+parameters.
 
 Windows
 -------
