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

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

Open
infominer33 opened this issue Jul 10, 2022 · 3 comments
Open

Comments

@infominer33
Copy link

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

@geraldb
Copy link
Member

geraldb commented Jul 10, 2022

Thanks for reporting the error. Cheers.

@infominer33
Copy link
Author

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

@geraldb
Copy link
Member

geraldb commented Jan 18, 2023

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!

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