itsToggle / plex_debrid

Plex torrent streaming through Debrid Services
1.42k stars 114 forks source link

Issues scraping indexers from jackett #19

Closed Bayharbourbutcher1 closed 2 years ago

Bayharbourbutcher1 commented 2 years ago

Hi there, here is the snip of the errors i a m getting when only jackett scraper is enabled. I have no issues using the default scrapers. Let me know if you need anything else. I am using a windows 10 machine. image

Bayharbourbutcher1 commented 2 years ago

image

itsToggle commented 2 years ago

Oh thats a wierd one! Usually Jackett returns either a magnet directly (e.g. rarbg indexer) or it provides a link that redirects to a magnet (e.g. 1337x). It seems that one of your indexers does not redirect to a magnet. I should catch that exception. Could you tell me which indexers you got set up?

Bayharbourbutcher1 commented 2 years ago

image These are the indexers i have configured.

itsToggle commented 2 years ago

Alright so the script will now catch the exception. I will try to find the root cause of the problem though. One of these indexers doesn't redirect to a magnet link in the way the others do, as I mentioned earlier. I'll keep this issue open until I fix the root cause of the problem, but you will be able to use Jackett now :)

Bayharbourbutcher1 commented 2 years ago

Thank you so much mate! Ill re-nable jackett again and test, will get back to you with the results.

Bayharbourbutcher1 commented 2 years ago

image So i copied over your changes to the script and re-enabled jackett, now getting the above error, any ideas?

itsToggle commented 2 years ago

alrighty, Ive added a catch for that exception aswell. This should never happen though, since plex should provide a release date for all listed seasons - and even if not, the object should absolutely have the attribute that the exception said was missing. would you mind telling me which show this was happening for?

Bayharbourbutcher1 commented 2 years ago

Hey thanks again for that! I am actually not sure which show this was for as i hadn't actually added anything new in the last 24 hours or so. Ill update the script now and test.

itsToggle commented 2 years ago

Oh than it might have been a new season of a watchlisted show that plex added while the script was running? Not sure.

Bayharbourbutcher1 commented 2 years ago

image Another error, copied over the changes to the script and ran again.

itsToggle commented 2 years ago

jesus im sorry, should be fixed now. Would you mind telling me which shows you have in the watchlist? It seems that one season of those shows is actually missing the release date, Id like to know what other attributes could be missing aswell :)

Bayharbourbutcher1 commented 2 years ago

No need to apologize mate, you're doing us a favor. I have the following in my watchlist: Virgin river Under the banner of heaven Ted lasso Severance Servant Prehistoric planet Night sky The lincoln lawyer The flight attendant Bosch legacy

Bayharbourbutcher1 commented 2 years ago

image Got the error again after updating the script but it has continued to run this time though.

itsToggle commented 2 years ago

Yeah thats intended, you have the ui-setting 'debug printing' enabled it seems :) I think I found the problematic season, looks like Servant - Season 4 has no metadata on its release date. Does the error keep popping up, or does it show just once?

Edit: seems that it just shows everytime the entire watchlist is refreshed, so every 30 minutes. I think that fixes that :)

Bayharbourbutcher1 commented 2 years ago

Fantastic - Thanks again, everything is working well now! Possible to add content size limit options in the config at all? I know i am getting a bit greedy now haha. Its just that sometime you dont want to add gigantic dolby vision 80gb rips which struggle to stream :)

itsToggle commented 2 years ago

Not at all haha, I very much apprechiate the ideas. The current sorting of releases is based on regex definitions. Ive looked into this before, and there is no easy way to write a size limit as a regex definition - so i might need to change up the release sorting a bit. This is a good excuse to make adding rules more intuitive aswell. I'll see what I can do :)