]> jfr.im git - z_archive/twitter.git/blobdiff - twitter/stream.py
Merge pull request #203 from RouxRC/pr-fix-timeout
[z_archive/twitter.git] / twitter / stream.py
index 77f86d01f49dd6f17b281e4da8f079e5f92480e5..6ecc49ed760a54591477274c0c1517556900d04e 100644 (file)
@@ -8,53 +8,97 @@ except ImportError:
 import json
 from ssl import SSLError
 import socket
-import sys
+import sys, select, time
 
 from .api import TwitterCall, wrap_response, TwitterHTTPError
 
+python27_3 = sys.version_info >= (2, 7)
+def recv_chunk(sock): # -> bytearray:
+
+    header = sock.recv(8)  # Scan for an up to 16MiB chunk size (0xffffff).
+    crlf = header.find(b'\r\n')  # Find the HTTP chunk size.
+
+    if crlf > 0:  # If there is a length, then process it
+
+        size = int(header[:crlf], 16)  # Decode the chunk size. Rarely exceeds 8KiB.
+        chunk = bytearray(size)
+        start = crlf + 2  # Add in the length of the header's CRLF pair.
+
+        if size <= 3:  # E.g. an HTTP chunk with just a keep-alive delimiter or end of stream (0).
+            chunk[:size] = header[start:start + size]
+        # There are several edge cases (size == [4-6]) as the chunk size exceeds the length
+        # of the initial read of 8 bytes. With Twitter, these do not, in practice, occur. The
+        # shortest JSON message starts with '{"limit":{'. Hence, it exceeds in size the edge cases
+        # and eliminates the need to address them.
+        else:  # There is more to read in the chunk.
+            end = len(header) - start
+            chunk[:end] = header[start:]
+            if python27_3:  # When possible, use less memory by reading directly into the buffer.
+                buffer = memoryview(chunk)[end:]  # Create a view into the bytearray to hold the rest of the chunk.
+                sock.recv_into(buffer)
+            else:  # less efficient for python2.6 compatibility
+                chunk[end:] = sock.recv(max(0, size - end))
+            sock.recv(2)  # Read the trailing CRLF pair. Throw it away.
+
+        return chunk
+
+    return bytearray()
+
+
 class TwitterJSONIter(object):
 
-    def __init__(self, handle, uri, arg_data, block=True):
-        self.decoder = json.JSONDecoder()
+    def __init__(self, handle, uri, arg_data, block=True, timeout=None):
         self.handle = handle
-        self.buf = b""
+        self.uri = uri
+        self.arg_data = arg_data
         self.block = block
+        self.timeout = timeout
+
 
     def __iter__(self):
-        if sys.version_info >= (3, 0):
-            sock = self.handle.fp.raw._sock
-        else:
-            sock = self.handle.fp._sock.fp._sock
+        sock = self.handle.fp.raw._sock if sys.version_info >= (3, 0) else self.handle.fp._sock.fp._sock
         sock.setsockopt(socket.SOL_SOCKET, socket.SO_KEEPALIVE, 1)
-        if not self.block:
-            sock.setblocking(False)
+        sock.setblocking(self.block and not self.timeout)
+        buf = ''
+        json_decoder = json.JSONDecoder()
+        timer = time.time()
         while True:
             try:
-                utf8_buf = self.buf.decode('utf8').lstrip()
-                res, ptr = self.decoder.raw_decode(utf8_buf)
-                self.buf = utf8_buf[ptr:].encode('utf8')
+                buf = buf.lstrip()
+                res, ptr = json_decoder.raw_decode(buf)
+                buf = buf[ptr:]
                 yield wrap_response(res, self.handle.headers)
                 continue
             except ValueError as e:
-                if self.block:
-                    pass
-                else:
-                    yield None
-            except urllib_error.HTTPError as e:
-                raise TwitterHTTPError(e, uri, self.format, arg_data)
-            # this is a non-blocking read (ie, it will return if any data is available)
+                if self.block: pass
+                else: yield None
             try:
-                self.buf += sock.recv(1024)
+                buf = buf.lstrip()  # Remove any keep-alive delimiters to detect hangups.
+                if self.timeout and not buf:  # This is a non-blocking read.
+                    ready_to_read = select.select([sock], [], [], self.timeout)
+                    if not ready_to_read[0] and time.time() - timer > self.timeout:
+                        yield {'timeout': True}
+                        continue
+                timer = time.time()
+                buf += recv_chunk(sock).decode('utf-8')
+                if not buf:
+                    yield {'hangup': True}
+                    break
             except SSLError as e:
-                if (not self.block) and (e.errno == 2):
-                    # Apparently this means there was nothing in the socket buf
-                    pass
-                else:
-                    raise
+                # Error from a non-blocking read of an empty buffer.
+                if (not self.block or self.timeout) and (e.errno == 2): pass
+                else: raise
+
+def handle_stream_response(req, uri, arg_data, block, timeout=None):
+    try:
+        handle = urllib_request.urlopen(req,)
+    except urllib_error.HTTPError as e:
+        raise TwitterHTTPError(e, uri, 'json', arg_data)
+    return iter(TwitterJSONIter(handle, uri, arg_data, block, timeout=timeout))
 
-def handle_stream_response(req, uri, arg_data, block):
-    handle = urllib_request.urlopen(req,)
-    return iter(TwitterJSONIter(handle, uri, arg_data, block))
+class TwitterStreamCallWithTimeout(TwitterCall):
+    def _handle_response(self, req, uri, arg_data, _timeout=None):
+        return handle_stream_response(req, uri, arg_data, block=True, timeout=self.timeout)
 
 class TwitterStreamCall(TwitterCall):
     def _handle_response(self, req, uri, arg_data, _timeout=None):
@@ -87,16 +131,19 @@ class TwitterStream(TwitterStreamCall):
     """
     def __init__(
         self, domain="stream.twitter.com", secure=True, auth=None,
-        api_version='1.1', block=True):
+        api_version='1.1', block=True, timeout=None):
         uriparts = ()
         uriparts += (str(api_version),)
 
         if block:
-            call_cls = TwitterStreamCall
+            if timeout:
+                call_cls = TwitterStreamCallWithTimeout
+            else:
+                call_cls = TwitterStreamCall
         else:
             call_cls = TwitterStreamCallNonBlocking
 
         TwitterStreamCall.__init__(
             self, auth=auth, format="json", domain=domain,
             callable_cls=call_cls,
-            secure=secure, uriparts=uriparts)
+            secure=secure, uriparts=uriparts, timeout=timeout, gzip=False)