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

end event never fired #413

Closed
kof opened this issue Jul 7, 2014 · 10 comments
Closed

end event never fired #413

kof opened this issue Jul 7, 2014 · 10 comments

Comments

@kof
Copy link
Contributor

kof commented Jul 7, 2014

When trying to send a get request to this site

http://www.channellock.com/blog/post/feed/2010/06/09/A-message-from-President-and-CEO-William-S-DeArment.aspx?format=ATOM

@kof
Copy link
Contributor Author

kof commented Jul 10, 2014

Here another example, this one redirects correctly, but then no data and no end events fired

http://www.tvcatalunya.com/rss/videos/tots_multimedies_182564611_rss.xml

or

http://blog.maartenballiauw.be/category/feed/C.aspx

@kof
Copy link
Contributor Author

kof commented Jul 11, 2014

@Naman34 wdyt?

@nmn
Copy link
Contributor

nmn commented Jul 17, 2014

Node or browser, or both? Sound like a bug similar to the redirect bug.

@kof
Copy link
Contributor Author

kof commented Jul 17, 2014

node v0.11.13 didn't test the older one

@nmn
Copy link
Contributor

nmn commented Jul 17, 2014

Could you test in a stable node? this could be a bug in Node core.

@kof
Copy link
Contributor Author

kof commented Jul 17, 2014

same on node 0.10

@nmn
Copy link
Contributor

nmn commented Jul 17, 2014

Alright. I'll try to find the bug. In the mean time, let me know if you find anything interesting.

@rase-
Copy link
Contributor

rase- commented Mar 1, 2015

Is this still an issue?

A piece of sample code could be useful.

@defunctzombie
Copy link
Contributor

Closing due to age.

@knownasilya
Copy link

Still an issue, see https://travis-ci.org/Strider-CD/strider-github/jobs/104789778#L299
Code is here: https://github.com/Strider-CD/strider-github/blob/master/lib/api.js#L134

Works in > 0.10 of node, but 0.10 is broken.

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

5 participants