Include video ID in ERROR: messages #393

Open
opened 2026-02-20 21:11:37 -05:00 by deekerman · 6 comments
Owner

Originally created by @kull0rk3x on GitHub (Nov 3, 2012).

Hi,

I was wondering if it is possible to include the video ID in the error messages, eg. turning lines like
ERROR: YouTube said: This video contains content from UMG, who has blocked it in your country on copyright grounds.
to
ERROR [videoIDhere]: YouTube said: This video contains content from UMG, who has blocked it in your country on copyright grounds.
to be able to know which video failed when letting youtube-dl work on a playlist.
I wrote a little script that utilizes youtube-dl and mplayer to playback videos, and if this was changed I could unblock a few videos on the fly (until then I would have to download the whole playlist again via a proxy).

Regards

Originally created by @kull0rk3x on GitHub (Nov 3, 2012). Hi, I was wondering if it is possible to include the video ID in the error messages, eg. turning lines like _ERROR: YouTube said: This video contains content from UMG, who has blocked it in your country on copyright grounds._ to _ERROR [videoIDhere]: YouTube said: This video contains content from UMG, who has blocked it in your country on copyright grounds._ to be able to know which video failed when letting youtube-dl work on a playlist. I wrote a little script that utilizes youtube-dl and mplayer to playback videos, and if this was changed I could unblock a few videos on the fly (until then I would have to download the whole playlist again via a proxy). Regards
Author
Owner

@FiloSottile commented on GitHub (Nov 3, 2012):

Yeah, might be useful, but meanwhile the id is on the precedent output line, no?

@FiloSottile commented on GitHub (Nov 3, 2012): Yeah, might be useful, but meanwhile the id is on the precedent output line, no?
Author
Owner

@kull0rk3x commented on GitHub (Nov 3, 2012):

Normally yes, but if you add --get-url --get-title (so that mplayer can do the steaming), its not.

@kull0rk3x commented on GitHub (Nov 3, 2012): Normally yes, but if you add --get-url --get-title (so that mplayer can do the steaming), its not.
Author
Owner

@kull0rk3x commented on GitHub (Feb 8, 2013):

This patch (to be applied on git-master) patches all errors & warnings to ERROR/WARNING: [InfoExtractor] video_id: error_message.

@kull0rk3x commented on GitHub (Feb 8, 2013): [This patch](http://pastebin.com/7CXbmMsz) (to be applied on git-master) patches all errors & warnings to _ERROR/WARNING: [InfoExtractor] video_id: error_message_.
Author
Owner

@kull0rk3x commented on GitHub (Mar 3, 2013):

I posted a solution to this problem, please consider applying the patch. If theres any more work needed for this, I'll update the diff.

@kull0rk3x commented on GitHub (Mar 3, 2013): I posted a solution to this problem, please consider applying the patch. If theres any more work needed for this, I'll update the diff.
Author
Owner

@yasoob commented on GitHub (Jun 24, 2013):

@kull0rk3x Can you do a pull request ? It will ease the developers work while applying the patch.

@yasoob commented on GitHub (Jun 24, 2013): @kull0rk3x Can you do a pull request ? It will ease the developers work while applying the patch.
Author
Owner

@dirkf commented on GitHub (Feb 5, 2024):

video_id can now be passed into ExtractorError. Code needs to be reviewed to pass that parameter where appropriate.

@dirkf commented on GitHub (Feb 5, 2024): `video_id` can now be passed into `ExtractorError`. Code needs to be reviewed to pass that parameter where appropriate.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/youtube-dl#393
No description provided.