-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add timeout config on settings #25
Comments
In jellyfin the timeout can't be longer than 30 seconds Line 85 in 9a7846d
|
But I can make provider timeout configurable, as it's currently 20 seconds subbuzz/Providers/Http/Download.cs Line 29 in 9a7846d
|
Disclaimer: I'm a sysadmin/network admin, so I'm not that well versed in coding (specially OO) so please point out anything I did wrong here, but I've forked the project, made a .yaml for 10.10 (basically a copy of the 10.9 only changing the version) and edited your workflow to build a 10.10 version of the plugin (basically changing 10.9 to 10.10), it gave me a dll that I replaced in the plugins folder, and then I restarted the Jellyfin server, I'm running it on Windows 10 22H2 (Build 19045.5131) as a service under the built-in Network Service account. I've tried forking the project and changing that variable as well as the one found in the Client.cs (ConnectTimeout) to 60 seconds, I'm running that build now, I've probably done something wrong as the query is still timing out at 10 seconds see log bellow:
|
Funny thing is that if (at least after the code ran so it could be cached) I run that URL on my browser it returns kinda quickly (certainly less than 10s) I don't know if it is this is a genuine timeout or something with the implementation of the provider. I can test any changes that you make to help with troubleshooting. |
[2024-12-04 00:14:19.979 -03:00] [INF] [18] subbuzz.Providers.PodnapisiNet: Search duration: 10,014 sec. Subtitles found: 0 ConnectTimeout is set to 10 seconds, meaning that the HTTP connection must be established in 10 seconds. subbuzz/Providers/Http/Client.cs Line 41 in 9a7846d
|
It would be nice if we could modify the http timeout in the settings, there are a lot of timeout errors in the logs, some sites are taking longer than 20s to respond:
[2024-11-29 00:45:32.047 -03:00] [ERR] [24] subbuzz.Providers.PodnapisiNet: GET: https://www.podnapisi.net/subtitles/search/old?sXML=1&sL=pt-br&sK=Steven+Universe&sTS=4&sTE=16: Search error: System.Threading.Tasks.TaskCanceledException: The operation was canceled. ---> System.TimeoutException: A connection could not be established within the configured ConnectTimeout. --- End of inner exception stack trace ---
The text was updated successfully, but these errors were encountered: