summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Panter <vadmium+py@gmail.com>2016-03-28 00:22:09 (GMT)
committerMartin Panter <vadmium+py@gmail.com>2016-03-28 00:22:09 (GMT)
commitf6b1d66a3ca060247e03e7b198db8a5a966b836e (patch)
tree09a7423be3c47ec27efb71f5457110a4a3361d1e
parentce913877e42b7fa03434c2e765ace891e0f5c4dc (diff)
downloadcpython-f6b1d66a3ca060247e03e7b198db8a5a966b836e.zip
cpython-f6b1d66a3ca060247e03e7b198db8a5a966b836e.tar.gz
cpython-f6b1d66a3ca060247e03e7b198db8a5a966b836e.tar.bz2
Issue #23804: Fix SSL recv/read(0) to not return 1024 bytes
-rw-r--r--Doc/library/ssl.rst2
-rw-r--r--Lib/ssl.py6
-rw-r--r--Lib/test/test_ssl.py9
-rw-r--r--Misc/NEWS3
4 files changed, 15 insertions, 5 deletions
diff --git a/Doc/library/ssl.rst b/Doc/library/ssl.rst
index 79b1a47..98e866f 100644
--- a/Doc/library/ssl.rst
+++ b/Doc/library/ssl.rst
@@ -842,7 +842,7 @@ SSL Sockets
SSL sockets also have the following additional methods and attributes:
-.. method:: SSLSocket.read(len=0, buffer=None)
+.. method:: SSLSocket.read(len=1024, buffer=None)
Read up to *len* bytes of data from the SSL socket and return the result as
a ``bytes`` instance. If *buffer* is specified, then read into the buffer
diff --git a/Lib/ssl.py b/Lib/ssl.py
index df39592..65ad38f 100644
--- a/Lib/ssl.py
+++ b/Lib/ssl.py
@@ -561,7 +561,7 @@ class SSLObject:
server hostame is set."""
return self._sslobj.server_hostname
- def read(self, len=0, buffer=None):
+ def read(self, len=1024, buffer=None):
"""Read up to 'len' bytes from the SSL object and return them.
If 'buffer' is provided, read into this buffer and return the number of
@@ -570,7 +570,7 @@ class SSLObject:
if buffer is not None:
v = self._sslobj.read(len, buffer)
else:
- v = self._sslobj.read(len or 1024)
+ v = self._sslobj.read(len)
return v
def write(self, data):
@@ -776,7 +776,7 @@ class SSLSocket(socket):
# EAGAIN.
self.getpeername()
- def read(self, len=0, buffer=None):
+ def read(self, len=1024, buffer=None):
"""Read up to LEN bytes and return them.
Return zero-length string on EOF."""
diff --git a/Lib/test/test_ssl.py b/Lib/test/test_ssl.py
index 8c0dd31..645ec8d 100644
--- a/Lib/test/test_ssl.py
+++ b/Lib/test/test_ssl.py
@@ -2792,13 +2792,20 @@ else:
# consume data
s.read()
- # read(-1, buffer) is supported, even though read(-1) is not
data = b"data"
+
+ # read(-1, buffer) is supported, even though read(-1) is not
s.send(data)
buffer = bytearray(len(data))
self.assertEqual(s.read(-1, buffer), len(data))
self.assertEqual(buffer, data)
+ # recv/read(0) should return no data
+ s.send(data)
+ self.assertEqual(s.recv(0), b"")
+ self.assertEqual(s.read(0), b"")
+ self.assertEqual(s.read(), data)
+
# Make sure sendmsg et al are disallowed to avoid
# inadvertent disclosure of data and/or corruption
# of the encrypted data stream
diff --git a/Misc/NEWS b/Misc/NEWS
index a855451..59c0828 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -101,6 +101,9 @@ Library
- Issue #26644: Raise ValueError rather than SystemError when a negative
length is passed to SSLSocket.recv() or read().
+- Issue #23804: Fix SSL recv(0) and read(0) methods to return zero bytes
+ instead of up to 1024.
+
- Issue #26616: Fixed a bug in datetime.astimezone() method.
- Issue #21925: :func:`warnings.formatwarning` now catches exceptions on