VLSub 9.13 HTTP time-outs

When i try to search for subtitles moments later ,appears a message asking to kill the extension and colse vlc,When i try open vlc apears saying that vlc just crash do you want to report.

Log files informations

Added attachment:

• log

The related forum discussion: https://forum.videolan.org/viewtopic.php?f=14&t=136182#p449683

The Extension needs more time. Please consider disabling the buggy Watch timer in /modules/lua/extension.c. It could be a simple solution.

Can you explain step by step how to disable watch timer?

Sorry, it is just a suggestion for some VLC developer reading this ticket and willing to look at the case.

Ok thanks i apreciate it,i will wait for future solutions.

Mederi bro i just try the tip from the link you give to me,and works if i wait a while like a minute +- it works fine,this has to be a bug,but i need a better solution to this problem.

Replying to [comment:1 mederi]:

The related forum discussion: https://forum.videolan.org/viewtopic.php?f=14&t=136182#p449683

The Extension needs more time. Please consider disabling the buggy Watch timer in /modules/lua/extension.c. It could be a simple solution.

What makes you think that the watchdog is buggy? It rather looks like it is working and the extension is taking an inordinate amount of time. Don't shoot the messenger; don't remove the canary: Fix the extension.

assigned to @linkfanel

Yeah and indeed it is a bug in VLSub. The HTTP code there was written with two left hands, and it broke after some server-side updates of OpenSubtitles.org.

changed title from Vlc sub 9.13 extension kill to VLSub 9.13 HTTP time-outs

added Component::LUA scripts label

I don't do lua extensions.

Replying to [comment:8 linkfanel]:

I don't do lua extensions.

Trac automatically assigns unassigned bugs to you when the component is changed to "Lua scrits". Nothing personal. You are free to deassign bugs.

unassigned @linkfanel

I know.

Replying to [comment:6 courmisch]:

What makes you think that the watchdog is buggy? It rather looks like it is working and the extension is taking an inordinate amount of time. Don't shoot the messenger; don't remove the canary: Fix the extension. Users report their new experience with "Extension not responding!" dialog box: click "Yes" > VLC crash / wait over a minute > VLSub results.

When the watchdog dialog box appears (Extension not responding!), usually users think that the extension crashed and needs to be closed. Perhaps the message could contain information that this does not necessarily mean a crash and probably the extension just needs more time to finish its task.

Sometimes, if users click the "Yes" button in the watchdog dialog box, the whole VLC disappears. Then users need to open VLC again and sometimes the "VLC crash reporting" dialog box appers.

It seems that there is a problem in communication between VLSub and opensubtitles.org server at the moment, but the watchdog crashing problem is still there for all Extensions with calculations/operations taking long time. If users do not touch anything and just wait more than a minute, the watchdog dialog box disappears and results appear in VLSub.

Added attachment:

• Watchdog dialog - VLC crash.png

Also #17566 (closed)

0001-lua-fix-VLSub-HTTP-timeouts-bug-17561.patch

Added attachment:

• 0001-lua-fix-VLSub-HTTP-timeouts-bug-17561.patch

I've got a quick fix for this bug (and one which does not imply to downgrade the HTTP header like in ticket 17566).

VLSub OpenSubtitles client rely upon the "Content-Length" header as a way to stop from receiving data. Problem here, OpenSubtitles use CloudFlare as CDN and always return gzipp'ed response with a "X-Uncompressed-Content-Length" header filled out. Since "contentLength" var is set to nil, there is here a infinite loop on polling which ends up freezing the extension.

The dirty way to fix the timeout problem is to look for both headers, while the correct way is to implement a reliable http client as well as a reliable xmlrpc library.