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

Jackett index timeout causes full timeout #119

Open
stryan opened this issue May 28, 2021 · 3 comments
Open

Jackett index timeout causes full timeout #119

stryan opened this issue May 28, 2021 · 3 comments

Comments

@stryan
Copy link

stryan commented May 28, 2021

This is more of an FYI then a full Issue report, but I figured I'd let people know in case it is a full issue.

If one of the indexers in Jackett takes too long (over 60 seconds) to respond, it will result in all querys to Jackett timing out. This occurs even if there are other Indexers available and/or the misbehaving indexer is unneeded for the query.

This is difficult to notice, as Nefarious will only report a timeout error contacting the Jacket server entirely. Validating the Indexers under settings doesn't show the error, as the page will lock up until the timeout occurs with no error messages, and the settings section will look fine since the other parts respond correctly.

@lardbit
Copy link
Owner

lardbit commented May 28, 2021

You're right, and this is due to the fact nefarious is taking advantage of jackett's aggregate indexer feature https://github.com/Jackett/Jackett#aggregate-indexers which has the drawback of the whole request fails if a single indexer fails. I chose the aggregate option out of convenience but maybe it's time to reconsider and just query each indexer individually and then comb throttle results afterwards to find the best match.

I'll keep this issue open until I have time to work on it. For the time being, the settings page should issue an error, though, so I'll try and fix that first.

Thanks for pointing this out. It's bit me a few times too and I had to remove the failing indexer to resolve.

@lardbit
Copy link
Owner

lardbit commented May 31, 2021

For the meantime, I fixed the Settings page to at least display an error of which Jackett indexer failed.

image

@stryan
Copy link
Author

stryan commented Jun 1, 2021

Great! Honestly, that's all I needed.

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