MycroftAI / skill-npr-news

Mycroft AI official News Skill, providing the latest news report from your favorite broadcast.
https://mycroft.ai/skills
Apache License 2.0
9 stars 42 forks source link

rss feed - bad assumption #101

Closed JarbasAl closed 2 years ago

JarbasAl commented 4 years ago

if a stream does not end with .mp3 it assumes it is a rss feed

This is a bad assumption, some streams are static and don't end with an extension

workaround: https://github.com/MycroftAI/skill-npr-news/pull/102/files#diff-8e2c1f727276d30ced72fa67cb848772a9802162bec39b42a8df5359fd063a4cR321

krisgesling commented 2 years ago

Closing as the Skill now uses feedparser to test if a url is an RSS feed.