Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

stream: Include _timeout parameter in _handle_response #106

Merged
merged 1 commit into from Dec 11, 2012

Conversation

Projects
None yet
2 participants
Contributor

edavis commented Dec 6, 2012

In a5aab11 (and renamed in 8fd7289), _handle_response began receiving
a _timeout parameter when invoked via TwitterCall.call.

The _handle_response method in api.py was updated to reflect this, but
not the two (TwitterStreamCall, TwitterStreamCallNonBlocking) in
stream.py.

This commit adds _timeout to the two streaming classes to accommodate
the changes.

@edavis edavis stream: Include _timeout parameter in _handle_response
In a5aab11 (and renamed in 8fd7289), _handle_response began receiving
a _timeout parameter when invoked via TwitterCall.__call__.

The _handle_response method in api.py was updated to reflect this, but
not the two (TwitterStreamCall, TwitterStreamCallNonBlocking) in
stream.py.

This commit adds _timeout to the two streaming classes to accommodate
the changes.
9e19b5a

@sixohsix sixohsix added a commit that referenced this pull request Dec 11, 2012

@sixohsix sixohsix Merge pull request #106 from edavis/add-timeout-to-stream
stream: Include _timeout parameter in _handle_response
In a5aab11 (and renamed in 8fd7289), \_handle_response began receiving
a \_timeout parameter when invoked via TwitterCall.__call__.

The _handle_response method in api.py was updated to reflect this, but
not the two (TwitterStreamCall, TwitterStreamCallNonBlocking) in
stream.py.

This commit adds _timeout to the two streaming classes to accommodate
the changes.
f3e81f3

@sixohsix sixohsix merged commit f3e81f3 into sixohsix:master Dec 11, 2012

Owner

sixohsix commented Dec 11, 2012

Thank you for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment