X-Git-Url: https://jfr.im/git/z_archive/twitter.git/blobdiff_plain/741507400802fcda26c684a650cc66375fa4906e..86318060d2ca6236785b588ff0b65d3311bdfd5b:/twitter/stream.py

diff --git a/twitter/stream.py b/twitter/stream.py
index 2fd9cd8..b11a893 100644
--- a/twitter/stream.py
+++ b/twitter/stream.py
@@ -7,20 +7,29 @@ except ImportError:
     import urllib2 as urllib_error
 import json
 from ssl import SSLError
+import socket
+import sys, select, time
 
-from .api import TwitterCall, wrap_response
+from .api import TwitterCall, wrap_response, TwitterHTTPError
 
 class TwitterJSONIter(object):
 
-    def __init__(self, handle, uri, arg_data, block=True):
+    def __init__(self, handle, uri, arg_data, block=True, timeout=None):
         self.decoder = json.JSONDecoder()
         self.handle = handle
         self.buf = b""
         self.block = block
+        self.timeout = timeout
+        self.timer = time.time()
+
 
     def __iter__(self):
-        sock = self.handle.fp._sock.fp._sock
-        if not self.block:
+        if sys.version_info >= (3, 0):
+            sock = self.handle.fp.raw._sock
+        else:
+            sock = self.handle.fp._sock.fp._sock
+        sock.setsockopt(socket.SOL_SOCKET, socket.SO_KEEPALIVE, 1)
+        if not self.block or self.timeout:
             sock.setblocking(False)
         while True:
             try:
@@ -28,6 +37,7 @@ class TwitterJSONIter(object):
                 res, ptr = self.decoder.raw_decode(utf8_buf)
                 self.buf = utf8_buf[ptr:].encode('utf8')
                 yield wrap_response(res, self.handle.headers)
+                self.timer = time.time()
                 continue
             except ValueError as e:
                 if self.block:
@@ -38,34 +48,48 @@ class TwitterJSONIter(object):
                 raise TwitterHTTPError(e, uri, self.format, arg_data)
             # this is a non-blocking read (ie, it will return if any data is available)
             try:
-                self.buf += sock.recv(1024)
+                if self.timeout:
+                    ready_to_read = select.select([sock], [], [], self.timeout)
+                    if ready_to_read[0]:
+                        self.buf += sock.recv(1024)
+                        if time.time() - self.timer > self.timeout:
+                            yield {"timeout":True}
+                    else:
+                        yield {"timeout":True}
+                else:
+                    self.buf += sock.recv(1024)
             except SSLError as e:
-                if (not self.block) and (e.errno == 2):
+                if (not self.block or self.timeout) and (e.errno == 2):
                     # Apparently this means there was nothing in the socket buf
                     pass
                 else:
                     raise
 
-def handle_stream_response(req, uri, arg_data, block):
+def handle_stream_response(req, uri, arg_data, block, timeout=None):
     handle = urllib_request.urlopen(req,)
-    return iter(TwitterJSONIter(handle, uri, arg_data, block))
+    return iter(TwitterJSONIter(handle, uri, arg_data, block, timeout=timeout))
+
+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):
+    def _handle_response(self, req, uri, arg_data, _timeout=None):
         return handle_stream_response(req, uri, arg_data, block=True)
 
 class TwitterStreamCallNonBlocking(TwitterCall):
-    def _handle_response(self, req, uri, arg_data):
+    def _handle_response(self, req, uri, arg_data, _timeout=None):
         return handle_stream_response(req, uri, arg_data, block=False)
 
 class TwitterStream(TwitterStreamCall):
     """
-    Interface to the Twitter Stream API (stream.twitter.com). This can
-    be used pretty much the same as the Twitter class except the
-    result of calling a method will be an iterator that yields objects
-    decoded from the stream. For example::
+    The TwitterStream object is an interface to the Twitter Stream API
+    (stream.twitter.com). This can be used pretty much the same as the
+    Twitter class except the result of calling a method will be an
+    iterator that yields objects decoded from the stream. For
+    example::
 
-        twitter_stream = TwitterStream(auth=UserPassAuth('joe', 'joespassword'))
+        twitter_stream = TwitterStream(auth=OAuth(...))
         iterator = twitter_stream.statuses.sample()
 
         for tweet in iterator:
@@ -74,22 +98,25 @@ class TwitterStream(TwitterStreamCall):
     The iterator will yield tweets forever and ever (until the stream
     breaks at which point it raises a TwitterHTTPError.)
 
-    The `block` paramater controls if the stream is blocking. Default
+    The `block` parameter controls if the stream is blocking. Default
     is blocking (True). When set to False, the iterator will
     occasionally yield None when there is no available message.
     """
     def __init__(
         self, domain="stream.twitter.com", secure=True, auth=None,
-        api_version='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)