Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The oembed URL used in this module is using http://soundcloud.com/oembed but needs to be updated to https://soundcloud.com/oembed. The response from soundcloudfield_curl_get()
is returning an HTML redirection notice and causes the player to not work at all. The path needs to be updated to https://.
<html><body>You are being <a href="https://soundcloud.com/oembed?iframe=true&url=<removed>">redirected</a>.</body></html>
Comment | File | Size | Author |
---|---|---|---|
#3 | soundcloud-oembed_https-3075203-2-D8.patch | 859 bytes | jeffschuler |
#2 | soundcloudfield-3075203-1.patch | 1.03 KB | vinmassaro |
Comments
Comment #2
vinmassaro CreditAttribution: vinmassaro commentedComment #3
jeffschulerSame thing for D8.
Comment #4
alisonb CreditAttribution: alisonb commentedThanks so much for this. I made the change to https manually on a D7 site and it worked perfectly.
Comment #5
vinmassaro CreditAttribution: vinmassaro commentedTrivial change, moving to RTBC.
Comment #6
Catalyst_Red CreditAttribution: Catalyst_Red commentedThank you for sharing the fix!
Comment #7
dqdRTBC from me.
Comment #8
pingevt CreditAttribution: pingevt commentedWorked for me as well
Comment #9
msypes CreditAttribution: msypes commentedWorks for me too!
Thanks.
Comment #10
Chris Matthews CreditAttribution: Chris Matthews as a volunteer and at City of Oaks Design commentedIs there still a maintainer for the SoundCloud field module? The last commit on 8.x-1.x was in April 2017. It would be nice if the patch in #3 could be committed and pushed to a new release.
Comment #11
david.qdoscc CreditAttribution: david.qdoscc commented+1. This module is unusable without this one character patch. Please can someone commit it to save us all wasting time searching for the solution?
Comment #12
jeffschulerLooks like an issue to discuss finding new project maintainers was just opened -- #3138560: Module co-maintainers -- (anyone?) and @Webbeh reached out to the current maintainer.
See also Dealing with unsupported (abandoned) projects.
Comment #13
dqdDo not rush with such suggestions please. This project is not abandoned nor unsupported. MAROQQO digital media (the company I am CEO of) is willing to support the project development and has contacted Attila already. He is an awesome long-term contributor to the Drupal project and we offer co-maintainerchip regarding and alongside the new Drupal 8./9. contribution guideline initiative and automated deprecation patches etc.. So please have some more patience. The RTBC patches can be added quickly and if Attila agrees we can roll out Beta and final D8 release asap. But before that all patches need another review and we need to mark the project D9 ready.
Such comments are really not very useful nor do they add anything positive to the project. There is always a good reason for simple patches not to be added yet. So please let us respect the hard work already put into the projects and provide constructive and really helpful support.
Comment #14
bubuThanks for the patch #3. Works well.
Comment #15
benjarlett CreditAttribution: benjarlett commenteddoesnt work in combination with this patch (required for drupal 9) https://www.drupal.org/project/soundcloudfield/issues/3168341
Comment #18
attila.fekete CreditAttribution: attila.fekete commentedThank you everyone, merged into 7.x and 8.x dev branches.
I apologize for being away so long.