search

aquelemiguel / parrot

🦜 A hassle-free, highly performant, self-hosted Discord music bot with YouTube and Spotify support. Powered by yt-dlp.
MIT License
136 stars 44 forks source link

fix: ignore yt-dlp warnings #245

Closed SimonRask closed 1 year ago

SimonRask commented 1 year ago

The warnings interfere with JSON parsing

Fixes #242

lordzsolt commented 1 year ago

Great.... I just wasted half an afternoon reaching the same conclusion, and was about to open a PR 😅

@aquelemiguel Would be great to have this merged. As described in #242, yt-dlp will throw warnings like "HTTP 429 Too many requests" into stderr and stdout.

This breaks all the parsing logic that relies on NEWLINE_BYTE, because the output now consists of multiple lines.

joao-conde commented 1 year ago

Sorry for the delay, this makes absolute sense. Nasty bug you guys found. Thanks for the fix.