Commit graph

12 commits

Author SHA1 Message Date
DarkCat09 9071017dbf
Refactor: unify YDL object and proxy cfg field
This would allow to add sites much simplier
or even drop the ydl_fn_keys.
Main purpose of the refactoring is a code cleanup.
2024-05-28 13:21:23 +04:00
DarkCat09 7a75805423
Add SoundCloud 2024-05-28 11:10:50 +04:00
DarkCat09 1e1fe6dd16
Config: use None if no proxy specified 2024-05-28 09:09:59 +04:00
DarkCat09 45e5bcee27
Fix: del playlist_items parameter only when set
This code sometimes had raised KeyError
2024-05-08 13:54:56 +04:00
DarkCat09 247bf65439
Add YDL logging, change schema 2024-05-08 13:52:36 +04:00
DarkCat09 d0ed29e685
Optimize extract_info: no process=True for YouTube
For YT, processing separate videos greatly increases extract_info time,
but unnecessary when obtaining only track title.

For Yandex, processing is much faster, and we NEED it to get title.
2024-05-06 21:07:19 +04:00
DarkCat09 62ebecc87f
Remove path_length config option
I've misunderstood the FS limits, 255 is for filename, not path.
Path is limited only by libc, it's 4096.
The default outtmpl in musicdlp contains slashes, so it's a path.
But YDL considers it to be a filename, so the whole
outtmpl formatting result is trimmed to path_length.
Do we really need this? I think there are no "malicious" long-named tracks :)
2024-05-06 19:43:32 +04:00
DarkCat09 44fc039ac9
Refactor: merge ydl_wrap with ydl_pool.Downloader, add callbacks 2024-05-05 19:24:52 +04:00
DarkCat09 96be306b2b
More config options (tmpl, yt_proxy) + notes 2024-05-04 10:34:18 +04:00
DarkCat09 df242e833f
Config, cookies 2024-05-03 20:57:20 +04:00
DarkCat09 93c5ea3b18
Code cleanup
(there was an attempt to redirect yt-dlp stdout)
2024-05-03 20:10:42 +04:00
DarkCat09 2b47b002e6
Working WS server, yt-dlp objects "pool", async wrappers 2024-05-03 18:42:53 +04:00