DobyTang / LazyLibrarian

This project isn't finished yet. Goal is to create a SickBeard, CouchPotato, Headphones-like application for ebooks. Headphones is used as a base, so there are still a lot of references to it.
729 stars 72 forks source link

Getting Error Unhandled exception in search_rss_book about every 20 minutes in logs #1628

Closed mstrdraco closed 5 years ago

mstrdraco commented 5 years ago

To help with identifying and fixing issues, please include as much information as possible, including:

LazyLibrarian version number 50ddefedf211e9ba95f8eab5b8be1518cda475f0

Operating system used windows

Interface in use ( bookstrap)

Which api (Goodreads)

Source of your LazyLibrarian installation (github zip)

Relevant debug log with api keys and any passwords redacted

getting the following about every 20 minutes.

debug.zip

2018-10-18 12:44:14 ERROR Unhandled exception in search_rss_book: Traceback (most recent call last): File "C:\ProgramData\LazyLibrarian\lazylibrarian\searchrss.py", line 302, in search_rss_book resultlist, nproviders = IterateOverRSSSites() ValueError: too many values to unpack

Please note - usually a single line of log is not sufficient. The lines just before the error occurs can give useful context and greatly assist with debugging.

There is a built-in debug log creator on the logs page which makes it easy to provide this information

mstrdraco commented 5 years ago
2018-10-18 13:04:14 ERROR SEARCHALLRSS searchrss.py search_rss_book 370 Unhandled exception in search_rss_book: Traceback (most recent call last): File "C:\ProgramData\LazyLibrarian\lazylibrarian\searchrss.py", line 302, in search_rss_book resultlist, nproviders = IterateOverRSSSites() ValueError: too many values to unpack
jcal22 commented 5 years ago

Line 565 in providers.py is trying to return three variables:

return resultslist, providers, ''.join(set(dltypes))

Line 302 in searchrss.py is only looking for two variables.

philborman commented 5 years ago

The 3rd param isn't needed there, should have a _ Will be fixed shortly, just adding some other changes