dirkf

Results 1755 comments of dirkf

I like the idea of automated testing but CodeQL was proposed before and, looking at the results linked above, I think the [response still applies](https://github.com/ytdl-org/youtube-dl/pull/31952#issuecomment-1492989631). By all means implement this...

You can feel free to ignore/close off the issues that were flagged. If this was enabled for PRs, I'd only want to flag issues found in the new/changed code. With...

You can specify a commit instead of a branch, or `f'https://github.com/ytdl-org/youtube-dl/archive/{commit}.zip'`, but probably better to use a tagged nightly build, like https://github.com/ytdl-org/ytdl-nightly/archive/refs/tags/2024.03.13.zip.

Nonetheless, I have a branch that handles playlists, searches and channels and will get posted when time allows.

> [!CAUTION] > This comment may contain links to malicious content. **DO NOT** follow any links below Work-around: manually change the `/fo/` to `/fi/`. Fix (let's guess that `o` is...

Indeed, 4.2MiB upstream: an extreme portrait clip -- "turn the phone round !" -- of a woman with her "Home" book.

When I applied the change in ytdl-org/youtube-dl#32710, I aligned the original code with the latest version of `_extract_mpd_formats...()` from _yt-dlp_ to make porting easier. However #6654 landed almost immediately. Once...

So, https://www.discoveryplus.com/video/homestead-rescue/lone-wolf. I suppose it would be good to confirm that the _yt-dlp_ extractor detects the DRM, or otherwise.

The page in the UK is https://www.discoveryplus.com/gb/video/homestead-rescue/lone-wolf. Similar to the URL that I proposed, the _yt-dlp_ result is `This video is only available for registered users ...`. Is the problem...