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

Failure updating http://planet.python.org/rss20.xml #2

Closed
ionrock opened this issue Sep 6, 2013 · 3 comments
Closed

Failure updating http://planet.python.org/rss20.xml #2

ionrock opened this issue Sep 6, 2013 · 3 comments

Comments

@ionrock
Copy link

ionrock commented Sep 6, 2013

I get this error message when trying to update the Planet Python feed:

Elfeed update failed for http://planet.python.org/rss20.xml: (wrong-type-argument arrayp nil)

The URL in my elfeed-feeds is "http://planet.python.org/rss20.xml"

What happens is that when the title is clean (via elfeed-cleanup) it fails b/c it is passed nil.

@skeeto
Copy link
Owner

skeeto commented Sep 6, 2013

Thanks, these are exactly the reports I'm expecting as people come across feeds broken in ways I wasn't expecting. 👍 I believe I just fixed this in cebb66b and def2966.

It was actually the id variable that was nil because one of the RSS items has both empty link and guid tags. An empty guid is a sign of a bug on their end (though it would still be a valid RSS feed if they set the isPermanlink attribute to false). If they publish two RSS items with empty guids then RSS readers are supposed to treat the items as if they're the same exact item, which is probably not what they intend. For Elfeed, I'm treating empty guid tags as if if didn't exist and choosing something else to use as the item ID.

@skeeto
Copy link
Owner

skeeto commented Sep 13, 2013

I'm going to close this issue since I believe it's been fixed for a week now.

@skeeto skeeto closed this as completed Sep 13, 2013
@ionrock
Copy link
Author

ionrock commented Sep 13, 2013

Thanks for fixing this! It is working for me just fine.

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

2 participants