[vlc-devel] [PATCH 2/4] ytdl: access module for YoutubeDL

[Rémi Denis-Courmont]

Hi,

This runs YoutubeDL only once per URL. You can add however many handlers you want. There's already hundreds of them.

What you describe is a maybe problem for the Lua parser that runs each and every Lua handler script separately. Not a problem for this patchset.

Le 22 septembre 2020 08:56:17 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2020-09-22 7:50, Steve Lhomme wrote:
>> On 2020-09-21 18:02, Rémi Denis-Courmont wrote:
>>> This passes every HTTP(S) URL through YoutubeDL to extract playlists
>> 
>> That sounds like a terrible idea. What is the extra time and memory 
>> consumption added to every HTTP call ?
>
>Other aspect of this, if one wants to add vimeo, dailymotion, tiktok, 
>twitter, etc. Are we going to add one python layer for each ? Or should
>
>we centralize all these tests in one call ?
>
>> Also that means the python script becomes a very sensitive part of
>VLC. 
>> Is it going to be fuzzed ? What about the Youtube-DL that users may 
>> download from questionable locations, since we don't ship it (nor
>fuzz 
>> it) ?
>> 
>>> or media. If YoutubeDL fails, it falls back to the normal HTTP(S)
>>> access. This includes if YoutubeDL or Python 3 are not found.
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200922/00ba89e0/attachment.html>