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

TwitterStream not raising TwitterHTTPError #136

Closed
jcollins-va opened this issue Apr 7, 2013 · 2 comments
Closed

TwitterStream not raising TwitterHTTPError #136

jcollins-va opened this issue Apr 7, 2013 · 2 comments

Comments

@jcollins-va
Copy link

This may not be a bug, but perhaps a misunderstanding of how Twitter works with this library.

For a single OAuth access token, if I create one filter stream and then a second one, Twitter stops sending for the first stream. Twitter docs (https://dev.twitter.com/docs/streaming-apis/streams/public) say:

Each account may create only one standing connection to the public endpoints, 
and connecting to a public stream more than once with the same account credentials 
will cause the oldest connection to be disconnected.

However, I am never seeing the TwitterHTTPError raised on my first TwitterStream iterator.

Without this raise, I'm unsure as to how I can tell if a particular stream is dead or not?

@sixohsix
Copy link
Collaborator

sixohsix commented Apr 7, 2013

Good catch. It definitely should raise an error on close. I will have to look into this (but I'm not sure when I'll get to it.)

RouxRC added a commit to RouxRC/twitter that referenced this issue Dec 27, 2013
The existing error was never reached, luckily since it was using
undefined arguments. moved it in its right place and added handling
ssl errors as TwitterHTTPError.

(Fix python-twitter-tools#136)
scalebuilder pushed a commit to scalebuilder/twitter that referenced this issue Apr 18, 2014
The existing error was never reached, luckily since it was using
undefined arguments. moved it in its right place and added handling
ssl errors as TwitterHTTPError.

(Fix python-twitter-tools#136)
@RouxRC
Copy link
Member

RouxRC commented Jul 22, 2014

I think this could be closed now the stream logic was entirely redone, @jcollins-va can you confirm so?

@RouxRC RouxRC closed this as completed Dec 24, 2014
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

No branches or pull requests

3 participants