-
Notifications
You must be signed in to change notification settings - Fork 4
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
Sonarr dump-config: 'ValueError' for empty fields in SABnzbd settings #50
Comments
Hi there, sorry for taking a while to respond (was on break). Thanks for the bug report. This should be an easy fix, just need to reconfigure how Buildarr handles this field when the value is not set on the Sonarr instance. I'll make a PR for fixing this in the next release. |
Thanks for waiting. The fix for this issue has been released as:
Please give the new version a try to check that it resolves your issue. |
Hi, I'm using the docker container and when specifying latest, with an updated image, I am still getting this error for Sonarr, but not Radarr.
There is also a urlbase value for Host settings on the general tab which is blank for me? Or am I doing something else wrong? |
Hi there, sorry to hear you're still having issues. I have a couple of questions:
|
Sorry for the late reply, I am unable to replicate the issue to confirm it has been fixed since I have set a value and then deleted it, effectively making Sonarr report a default value instead of missing values. But I do believe this update would have fixed the issue since it is the same as the previously mentioned issue. Thank you so much for the time and effort you have put into the project! |
@Callum027 After a little sleep the problem became obvious- I have both a Sab and NZBget client configured in Sonarr. Deleting the NZBget config that I wasn't using, but also had an empty urlBase value, resulted in a successful dump. Here's a couple chunks of the log output before I solved it. If i can provide anything else to make it easier to fix for others let me know:
|
Thanks for the report @rabidoyster. It's likely that all of the indexers that can have a URL base supplied suffer from the same issue. This problem (and other related API parsing issues) will be resolved in the future by switching the Sonarr plugin to the Sonarr client Python library. For now, I'll make a separate issue for enabling |
Hi, this is an issue related to #39 where the default blank "Url Base" is not present for SABnzbd, causing an error during a config dump. This issue is also fixed by setting a value in Sonarr and clearing it.
The text was updated successfully, but these errors were encountered: