Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update streaming.py #694

Closed
wants to merge 1 commit into from
Closed

Conversation

theseusmedia
Copy link

if buf.read_line() returns none, then buf.read_line().strip() obviously fails

this feels like a kludge but it seems to work, and working is better than not working so maybe this is a useful pull request


edit: this might be fixed by PR#652 but I'm not sure

if buf.read_line() returns none, then buf.read_line().strip() obviously fails

this feels like a kludge but it seems to work, and working is better than not working so maybe this is a useful pull request
@gnesher
Copy link

gnesher commented Feb 2, 2016

+1 for this, currently having to hack local tweepy :/

@czpython
Copy link

I'm getting this issue but find it hard to figure out exactly why based on the code in https://github.com/tweepy/tweepy/blob/master/tweepy/streaming.py#L165. Maybe the connection is closed?

@gnesher
Copy link

gnesher commented Feb 17, 2016

Tweepy error handling is just not very useful sadly :/ I I've seen this error when I had problems with my credentials / when hitting a rate limit - but the only way to see this would be to manually add debug code within tweepy

@karelbilek karelbilek mentioned this pull request Mar 8, 2016
@karelbilek
Copy link

Is anyone from this project here? This would be great.

@computermacgyver
Copy link

Was just going to create a PR to fix this same issue, but found this one and wanted to give it a boost. I have been running with a very similar modification for months with no issues. It would be great to merge this PR.

benfei pushed a commit to benfei/tweepy that referenced this pull request Oct 11, 2016
If the stream is closed while reading, act as if it was closed before reading began.

This resloves tweepy#576, resloves tweepy#698 and resolves tweepy#734.

Similar to tweepy#694, tweepy#780.
@Aaron1011
Copy link
Contributor

I've fixed this in c1eddf1.

@Aaron1011 Aaron1011 closed this Nov 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants