pluto icon indicating copy to clipboard operation
pluto copied to clipboard

comparison of NilClass with Time failed (only fails with other blogs)

Open infominer33 opened this issue 1 year ago • 3 comments

this is a fun one

This feed http://dustycloud.org/blog/index.xml produces this error *** error: comparison of NilClass with Time failed

but only fails when I run it together with all of the other feeds... run on its own with no other blogs in the .ini file, works fine.

This is the obnoxiously long debug log: https://github.com/Identosphere/identity-blogcatcher/raw/master/weirdbug.txt This is the site that causes fail not failing on its own. https://raw.githubusercontent.com/Identosphere/identity-blogcatcher/master/weirdnotbug.txt

not sure what I'm expecting here, but I spent so much time chasing this down, I couldn't not report

infominer33 avatar Jul 10 '22 07:07 infominer33

Thanks for reporting the error. Cheers.

geraldb avatar Jul 10 '22 09:07 geraldb

ok, I figured out finally this happens with a blog that doesn't have a published date.

I went to a sqlite browser and found the offending entries, but I do noticed they had other date formats saved in the database such as "updated"

so ideally pluto would check other possibilities, instead of failing when there is no published date

infominer33 avatar Jan 18 '23 07:01 infominer33

Thanks for the update. If possible can you post the web feed entry that would really help (including makeing a test case / unit so it won't happen again).

The date uses fallbacks - see https://github.com/feedreader/pluto/blob/master/pluto-models/lib/pluto/models/item.rb#L43 but sure there are way too many different dates so it is hard to say what date causes the problem. Maybe it is just a bug in the template (merge) and not in the feed fetcher. Sorry I can only speculate. All the best. Cheers. Prosit 2023!

geraldb avatar Jan 18 '23 12:01 geraldb