Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for discogs.com #4790

Closed
error800 opened this issue Jan 27, 2015 · 6 comments
Closed

Add support for discogs.com #4790

error800 opened this issue Jan 27, 2015 · 6 comments

Comments

@error800
Copy link

@error800 error800 commented Jan 27, 2015

Discogs has an embedded youtube player on artist and release pages. Would be cool to be able to download the files (and load as playlist into mpv ;))
Examples:
Release: http://www.discogs.com/Imatran-Voima-Commando/master/15295
Artist: http://www.discogs.com/artist/14858-Imatran-Voima

@naglis
Copy link
Collaborator

@naglis naglis commented Jan 27, 2015

Note: discogs seems to block old user agents, in our case: Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20100101 Firefox/10.0 (Chrome)

@error800
Copy link
Author

@error800 error800 commented Jan 27, 2015

Youtube video ids can be found easily in the embedded iframe:

<iframe id="youtube_player_placeholder" data-video-ids="s5e7CVlBs1I,eUpvJEbGz8w,Pg7_n730w7Y,PznniwG_51Q,8QaKFAaGQxo,2p4H5wB_PHs,ifHC_wCWnrA,R22BIVmgfMI,hjgM6b3gAPw,1Vv_yw9wfHA,qQXMcZMDGEE,oBr6_2qZngQ,exfnVLQPfig,qnEo8Fs3OgY,L5LQveHQBBk,uIUM7fzpoEo,cAIIhsZxCLo,dQ6EsOTy86Q,6jK2SvQR4aM,3dnqvDrzn98,b7XMFNe3lao,bCLZoEQ48OE" ...></iframe>
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Feb 25, 2015

We should maybe update the user agent to a more recent version, since probably there won't be any website that blocks it.

@error800
Copy link
Author

@error800 error800 commented Feb 25, 2015

Good idea.
Right now i have:

--user-agent "Mozilla/5.0 (X11; Linux x86_64; rv:35.0) Gecko/20100101 Firefox/35.0"

in ~/.config/youtube-dl/config

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Feb 26, 2015

It has been updated in 1831393, so you no longer need to change it in your config file.

@jaimeMF jaimeMF closed this Feb 26, 2015
@error800
Copy link
Author

@error800 error800 commented Feb 26, 2015

nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.