inputnoise/youclonedl
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix _meta_run: hold lock for entire yt-dlp execution, not just scheduling

Browse Source 
Previously the lock was released before _run(), so multiple threads could
fire yt-dlp processes simultaneously — completely defeating the rate limiter.
Now the lock is held through the subprocess call and released in finally.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Mattias Thall
2026-05-27 01:21:36 +02:00
parent 1b010d4081
commit 86e7648075
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
backend/services/ytdlp.py
View File

@@ -33,8 +33,10 @@ def _meta_run(args: list[str], timeout: int = 60) -> tuple[str, str, int]:
wait = _META_MIN_GAP - (now - _meta_last_call) wait = _META_MIN_GAP - (now - _meta_last_call)
if wait > 0: if wait > 0:
time.sleep(wait + random.uniform(0.5, 2.5)) time.sleep(wait + random.uniform(0.5, 2.5))
_meta_last_call = time.monotonic() try:
return _run(args, timeout=timeout) return _run(args, timeout=timeout)
finally:
_meta_last_call = time.monotonic()
def _parse_date(date_str: str | None) -> datetime | None: def _parse_date(date_str: str | None) -> datetime | None: