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

Feed guesser relative url #44

Closed
toxinu opened this Issue Mar 21, 2013 · 3 comments

Comments

Projects
None yet
2 participants
Owner

toxinu commented Mar 21, 2013

I've try the feed guesser you just pull requested @orva and it cant handle relative url like on my website.

The Reader object just try the get feed at http:///atom.xml I think.

So maybe it's just invalid to put relative path in this meta tag? I don't really know what's the standard about it.

Owner

toxinu commented Mar 21, 2013

And by the way, if I give http://socketubs.net/atom.xml, I have bad feed error.

Contributor

orva commented Mar 21, 2013

Ah, yeah. Totally forgot about relative urls, will fix that.

Second issue is because for some reason your nginx puts 'application/xml' content-type instead of 'application/atom+xml' into http headers. Probably just best to allow plain xml content type as well in FeedFinder, I think that is quite common misconfiguration.

@orva orva referenced this issue Mar 21, 2013

Merged

Feed finder fixes #46

Owner

toxinu commented Mar 21, 2013

Fantastic!
That's a thing I have really loved on Google Reader :)
Many thanks @orva .

@toxinu toxinu closed this Mar 22, 2013

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