search

marcopeocchi / yt-dlp-web-ui

A terrible web ui and RPC server for yt-dlp. Designed to be self-hosted.
GNU General Public License v3.0
818 stars 84 forks source link

Confusing log. #193

Open YamatoRyou opened 3 weeks ago

YamatoRyou commented 3 weeks ago

Version running 2024.08.06 marcobaobao/yt-dlp-webui:latest

Describe the bug Trying to download some age-restricted videos with the Template Editor by passing in the specific parameter --cookies /cookies/cookies-for-x.txt, it failed. Log: level=ERROR msg="failed to retrieve metadata" id=******** err="ERROR: [twitter] *********: NSFW tweet requires authentication. Use --cookies, --cookies-from-browser, --username and --password, --netrc-cmd, or --netrc (twitter) to provide account credentials\n"

I thought my cookies were invalid after seeing this, but it took me a long time to find out that it was related to the title length of the target webpage. In other words, the error message in the log does not match the actual error message.

Verification method:

  1. Enter the container and manually execute the command in the container: yt-dlp --cookies /cookies/cookies-for-x.txt -o "/downloads/%(title)s.%(ext)s" https://x.com/************/status/******************
  2. Result: [download] Destination: /downloads/**************.mp4 ERROR: Unable to download video: [Errno 36] Filename too long: '******************************'

To Reproduce n/a

Expected behavior n/a

Screenshots n/a

Additional context n/a

marcopeocchi commented 3 weeks ago

I take advantage from this issue also for address the problem of the metadata setter not using cookies.

The mismatch with logs came from a wrong error handling in the metadata setter.

Thanks for pointing it out!

LeisureLinux commented 6 days ago

the config file as said: log_path: /home/your_user/yt-dlp-webui-workingdir

But when I actually config it, this is not a "workingdir" , it should be the real log file name.