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

Provide --client-id argument to override hard-coded clientid (eg: for soundcloud extractor) #14546

Closed
maxux opened this issue Oct 20, 2017 · 2 comments

Comments

@maxux
Copy link

@maxux maxux commented Oct 20, 2017

Prerequisites

  • I've verified and I assure that I'm running youtube-dl 2017.10.15.1
  • At least skimmed through the README, most notably the FAQ and BUGS sections
  • Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other

Issue

Soundcloud extractor returns HTTP 401, after search on existing issues, it looks like CLIENT_ID on the code needs to be updated.

Related Issues: #11327 and #12376

I changed the CLIENT_ID to a client-id I own from soundcloud API and it works again. I guess the client-id hardcoded is expired/revoked/ban.

Feature request

It would be cool to add a --client-id argument to override hardcoded client-id. You can create your own API Client ID via https://soundcloud.com/you/apps and then use this client-id. This could provide an easy-way to bypass 401 when hardcoded client-id is not working anymore.

In addition, it could helps to debug/confirm issue related to client-id without modifying the code. This could not be limited to soundcloud.

@dstftw dstftw closed this in a26a3c6 Oct 20, 2017
@maxux
Copy link
Author

@maxux maxux commented Oct 20, 2017

@dstftw Hello, I think referring this issue on the commit and not closing it would have been better. This issue is mainly about the feature request, not only about the issue.

@jacksgt
Copy link

@jacksgt jacksgt commented Nov 1, 2017

Thanks for this fix! 👍
(I was on 2017.10.15, fixed in 2017.10.20)

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.

None yet
2 participants
You can’t perform that action at this time.