Skip to content

Update app.min.js#302

Open
SergeyLukyanov777 wants to merge 1 commit into
yumata:mainfrom
SergeyLukyanov777:patch-1
Open

Update app.min.js#302
SergeyLukyanov777 wants to merge 1 commit into
yumata:mainfrom
SergeyLukyanov777:patch-1

Conversation

@SergeyLukyanov777

Copy link
Copy Markdown

Added parse timeout 90 and 120 seconds

Added parse timeout 90 and 120 seconds
@yumata

yumata commented Apr 3, 2026

Copy link
Copy Markdown
Owner

If I'm not mistaken, the standard response time is 1 minute, after which the server terminates the connection. And this won't work via JavaScript. But I could be wrong; I need to find out first.

@SergeyLukyanov777

SergeyLukyanov777 commented Apr 3, 2026

Copy link
Copy Markdown
Author

You set the request timeout yourself: network$2.timeout(1000 * Storage.field('parse_timeout')). If the parser takes a long time to process it, an error message appears: torrent_error_connect.

Set the wait time to 1 second and see for yourself...

@yumata

yumata commented Apr 3, 2026

Copy link
Copy Markdown
Owner

I understand that, but the parser is on the server, and a server like Nginx has a default maximum wait time of 60 seconds, after which it terminates the connection. The browser can also terminate the connection itself if it doesn't receive at least one byte within 60 seconds. This requires a special connection with a wait time, at least that's what the GPT chat told me :D

@SergeyLukyanov777

SergeyLukyanov777 commented Apr 4, 2026

Copy link
Copy Markdown
Author

I have Jackett installed on a Windows system with more than 70 indexers, and the browser, when requesting it, quietly waits for a response for a minute or more. Lampa breaks the connection due to timeout...
The ability to specify a timeout greater than 60 seconds would solve this problem.

@yumata

yumata commented Apr 5, 2026

Copy link
Copy Markdown
Owner

Okay, I'll add it. For future reference, we need to do a PR on another branch, https://github.com/yumata/lampa-source

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