[BUG] Playlist won't download if there is an unavailable video in it.

[deadlycfx]

Describe the bug
Playlist won't download if there is an unavailable video in it. It doesn't skip the unavailable video and won't download anything.

To Reproduce
Just download a playlist with and unavailable video.

Expected behavior
Skip the unavailable video and download the rest of the videos.

Environment

• YoutubeDL-Material 4.2
• Docker tag: nightly

[GlassedSilver]

Which fork of yt-dl are you using in your setup? And can you possibly provide logs?

Sorry that I can't get back to you with more than the usual initial trouble-shooting keypoints here and that I reply this late. :/

[mattkbach]

Having the same issue trying to download a playlist with a video in the playlist that is unavailable. Using the latest nightly build (4.2). Tried both youtube-dl and youtube-dlp

https://www.youtube.com/playlist?list=PLZWD78nR1R0nxp-rSjsdtK2J1HErSmOQA

This is what the logs say:

2022-01-14T23:26:16.579Z DEBUG: Removing all records from: files with filter: {"sub_id":"9d016eaf-69dc-4bab-8e6f-57cdb67f8c06"}
2022-01-14T23:26:16.587Z DEBUG: Successfully removed records from files
2022-01-14T23:26:26.367Z VERBOSE: Subscription: getting videos for subscription ACI4 with args: --dump-json,-o,subscriptions/playlists/ACI4/%(uploader)s [%(uploader_id)s]/%(title)s [%(id)s].%(ext)s,-ciw,--write-info-json,--print-json,-f,bestvideo[ext=mp4]+bestaudio[ext=m4a]/mp4,--write-thumbnail,--no-clean-infojson
2022-01-14T23:27:22.179Z VERBOSE: Subscription: finished check for ACI4
2022-01-14T23:27:22.186Z ERROR: WARNING: [youtube:tab] YouTube said: INFO - 2 unavailable videos are hidden
WARNING: [youtube:tab] YouTube said: INFO - Unavailable videos will be hidden during playback
ERROR: [youtube] 2cDoT3gm2Zo: Private video. Sign in if you've been granted access to this video
ERROR: [youtube] 0SfyFOVnjg4: Private video. Sign in if you've been granted access to this video

[GlassedSilver]

Pinning this issue now for visibility. I don't really have an idea how to fix this atm and googling this for feedback in regard to yt-dlp and how you'd possibly handle the error there directly with args... I come up empty.

So if anyone smarter than me knows something I'd appreciate pointers or even better a PR. :)

Thank you for reporting the bug you two!

[GlassedSilver]

yt-dlp's README says that you can use this flag to not abort on errors:

--no-abort-on-error              Continue with next video on download
                                 errors; e.g. to skip unavailable videos in
                                 a playlist (default)

This might help, but it could be that YT-DLM may abort this on its own, not sure. Can you check with yt-dlp and passing this?

[GlassedSilver]

I think I hastily applied the response-needed label here and had it auto-close when I in fact believe we should at least consider including the --no-abort-on-error arg to the defaults for one-off downloads. Thoughts, @Tzahi12345?

[mikepaxton]

Curious if any resolution has been achieved to this issue?
As an example below is a playlist I'd like to download but it has three hidden videos which seems to be preventing me from doing so with the same error message as Sleepymatto posted above. I've tried adding the custom arg --no--abort-on-error to the subscription but still to no avail.
https://www.youtube.com/playlist?list=PLRNgjwKiAAEBGH5pHqCai-c5Atu_A4hOT

[Tzahi12345]

Hi @mikepaxton @deadlycfx, this appears to be a yt-dlp issue. Btw --no-abort-on-error is yt-dlp specific and on by default, but in all youtube-dl forks there's the --ignore-errors are (or -i for short) which we already use.

My guess is that in yt-dlp, --dump-json and --ignore-errors cannot be used in conjunction, that is: errors will be thrown anyways. The good news is that the necessary info to continue the download exists, just in a slightly different place (in err.stdout for those interested).

I updated #657 to include the following commit: d225e84

Should be fixed shortly when that PR closes. Thanks @mikepaxton @sleepymatto @deadlycfx for the report & info!!