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

+Kill orphaned downloads, +various small tweaks #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ManiMatter
Copy link

@ManiMatter ManiMatter commented Aug 3, 2023

My first PR ever... :)

Added functionality to kill orphaned downloads (downloads initiated by Movies/TV Shows that were deleted whilst download is in progress)

Added few refinements:

  • dealing with OK status code (200)) in make_api_delete
  • better format for time wait
  • change how asynchio is called (previous approach was deprecated)
  • specified in ReadMe how to include the network in which this tool has to be called so that it can communicate with Sonarr/Radarr
  • made logging a bit more easy to read
  • fixed inconsistency in variable (API_TIMEOUT)

@raphaelgrau
Copy link

raphaelgrau commented Jun 4, 2024

Hi @ManiMatter, I am using the script and it seems like it's working but I got an error, here are my logs :

2024-06-04 09:21:56,015 [INFO]: Running media-tools script
2024-06-04 09:21:56,015 [INFO]: Checking Sonarr queue...
2024-06-04 09:21:56,047 [INFO]: Processing Sonarr queue...
2024-06-04 09:21:56,048 [INFO]: Checking the status of My TV Show (2001) - Episode 01 - 1080p
2024-06-04 09:21:56,048 [INFO]: Removing stalled Sonarr download: My TV Show (2001) - Episode 01 - 1080p
2024-06-04 09:21:56,074 [ERROR]: Error making API request to http://192.168.1.11:8989/api/v3/queue/1339089113: Expecting value: line 1 column 1 (char 0)

I am not an expert in python so I was wondering if maybe you could tell me if this was something you fixed on your PR?

Thanks!

@ManiMatter
Copy link
Author

hi @raphaelgrau - You might want to check out https://github.com/ManiMatter/decluttarr/
After this PR never got merged, I created my own version of this script, which by now I deem is more advanced than this one. My guess is that the bug you are having you won't encounter over there.

@MattDGTL - I don't mean to make promotion here for my script - I hope you find the advancement of your script helpful as well.

@raphaelgrau
Copy link

raphaelgrau commented Jun 4, 2024

@ManiMatter thanks for the reply, it looks really cool! I will give it a try.

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

Successfully merging this pull request may close these issues.

2 participants