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

Avoid hardcoded window.location.protocol #16

Closed
jtomaszewski opened this issue Mar 4, 2015 · 6 comments
Closed

Avoid hardcoded window.location.protocol #16

jtomaszewski opened this issue Mar 4, 2015 · 6 comments

Comments

@jtomaszewski
Copy link

Currently in the SDK protocol is hardcoded (a lot of window.location.protocol references). This makes the SDK unusable in Apache Cordova - as apps there have a file:// protocol.

A simple fix replacing all window.location.protocol with f.e. SC.options.protocol would be sufficient.

@gillesdemey
Copy link

Currently experiencing the same issue using Electron (or Node WebKit for that matter).

@angusfretwell
Copy link

👍

@chrisrickard
Copy link

Same issue... quite a show stopper.

Any word on this?

👍👍👍

@gillesdemey
Copy link

I've been using my own simple module for now, but I'm happy to take a look at this and send a PR if they're being looked at (I see a few PR's open from years ago).

@bluedaniel
Copy link

Yeah seeing this as well with Electron

@angusfretwell
Copy link

Here's a patched copy of the minified API which I bundled as a node module: https://github.com/angusfretwell/soundcloud-cordova-sdk

Suppose I should remove Cordova from the name of the package since this problem isn't really specific to Cordova 😅

@deanh deanh closed this as completed Sep 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

6 participants