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

[generic/Soundcloud] Please add support Soundcloud playlists and for the embedded Soundcloud player #2843

Closed
anovicecodemonkey opened this issue May 1, 2014 · 1 comment

Comments

@anovicecodemonkey
Copy link

@anovicecodemonkey anovicecodemonkey commented May 1, 2014

There's already soundcloud.py to handle some Soundcloud URLs but there's no support for Soundcloud playlists (e.g. https://w.soundcloud.com/player/?url=http%3A%2F%2Fapi.soundcloud.com%2Fplaylists%2F4110309&color=ff0000&auto_play=false&show_artwork=true) or support for the embedded Soundcloud player in the generic extractor.

Example URL: http://www.tiltbrass.org/bpc-aug-0_scr-02/ (this is the URL for the embedded playlist above but there is also just embedded single songs on some pages, the SoundcloudIE should be able to handle all cases correctly).

I've noticed in some cases on different sites, the embed is done with an <object> tag but most of the time it's an <iframe>.

$ youtube-dl http://www.tiltbrass.org/bpc-aug-0_scr-02/
[generic] bpc-aug-0_scr-02: Requesting header
WARNING: Falling back on generic information extractor.
[generic] bpc-aug-0_scr-02: Downloading webpage
[generic] bpc-aug-0_scr-02: Extracting information
ERROR: Unsupported URL: http://www.tiltbrass.org/bpc-aug-0_scr-02/; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
@anovicecodemonkey anovicecodemonkey changed the title [Soundcloud/generic] Add support for embedded Soundcloud player [Soundcloud/generic] Please add support for embedded Soundcloud player May 1, 2014
@anovicecodemonkey anovicecodemonkey changed the title [Soundcloud/generic] Please add support for embedded Soundcloud player [generic] Please add support for embedded Soundcloud player May 1, 2014
@anovicecodemonkey anovicecodemonkey changed the title [generic] Please add support for embedded Soundcloud player [generic/Soundcloud] Please add support Soundcloud playlists and for the embedded Soundcloud player May 2, 2014
@phihag
Copy link
Contributor

@phihag phihag commented May 5, 2014

Thank you for the report. Should be fixed in 2014.05.05. As usual, type youtube-dl -U to update.

@jaimeMF jaimeMF closed this May 5, 2014
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
3 participants
You can’t perform that action at this time.