Podcast: "Naked scientist" isnt reciving a xmlUrl #63

Closed
richardwiden opened this Issue May 9, 2013 · 4 comments

Projects

None yet

2 participants

Owner

The feed doesn't provide an xmlUrl. That's why it isn't in the parsed output.

I would consider adding this (as it would be easy enough -- when the user gives the url to parseUrl, fill-in meta.xmlUrl if it's absent from the feed), but I'm getting ready to deprecate the ability to have feedparser fetch a url for you, at which point it will be impossible for feedparser to patch that in for you. So, I'm not inclined to change anything.

Feel free to tell me I'm wrong.

@danmactough danmactough closed this May 9, 2013

The does provide a "source" I havent read up enough on rss/atom/standards
to know if this is some kind of standard. Would it be a good idea to use
this?

On Thu, May 9, 2013 at 10:42 PM, Dan MacTough notifications@github.comwrote:

The feed doesn't provide an xmlUrl. That's why it isn't in the parsed
output.

I would consider adding this (as it would be easy enough -- when the user
gives the url to parseUrl, fill-in meta.xmlUrl if it's absent from the
feed), but I'm getting ready to deprecate the ability to have feedparser
fetch a url for you, at which point it will be impossible for feedparser to
patch that in for you. So, I'm not inclined to change anything.

Feel free to tell me I'm wrong.


Reply to this email directly or view it on GitHubhttps://github.com/danmactough/node-feedparser/issues/63#issuecomment-17688481
.

Owner

source is for a completely different purpose. But that's a pretty clever fallback to check when the feed fails to provide the xmlUrl. 😄

xref #64

@danmactough danmactough reopened this May 9, 2013

I´m nowhere near good enough on node or js in general to make any valuable
contribution yet so I´d "happily leave the fixing to you" if that's okay.
I´ll be forced to look into it at some point but I´d prefer to keep
learning for now :)

On Thu, May 9, 2013 at 10:50 PM, Dan MacTough notifications@github.comwrote:

source is for a completely different purposehttp://cyber.law.harvard.edu/rss/rss.html#ltsourcegtSubelementOfLtitemgt.
But that's a pretty clever fallback to check when the feed fails to provide
the xmlUrl. [image: 😄]

xref #64 #64


Reply to this email directly or view it on GitHubhttps://github.com/danmactough/node-feedparser/issues/63#issuecomment-17688954
.

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