search

SpiderStrategies / node-tweet-stream

Node twitter module to hook into the public filter streaming, seamlessly updating the tracking keywords.
210 stars 43 forks source link

Fix bug with network latency #7

Closed dotcypress closed 10 years ago

nathanbowser commented 10 years ago

Heyo, sorry for the delay. Any tips on how I can reproduce the initial bug w/ a test?

nathanbowser commented 10 years ago

In other words, I'll totally write one if you can describe what you were seeing.

dotcypress commented 10 years ago

Maybe it depends on my network infrastructure, but for me this bug exists in 100% of requests. When stream fire 'data' event, i see only chunk of twit and JSON parser can't parse uncompleted JSON.

nathanbowser commented 10 years ago

Oh, right. I see the problem. I have an idea.

nathanbowser commented 10 years ago

Fixed in https://github.com/SpiderStrategies/node-tweet-stream/commit/9759eb15d2d97c1375676c67ccd0823750334816 with a test.

nathanbowser commented 10 years ago

Also, 1.3.2 is on npm. Let me know if this works for you, @dotcypress

dotcypress commented 10 years ago

Fixed. :+1: