Make the Dirble API token configurable and use HTTPS

notandy / ympd

Standalone MPD Web GUI written in C, utilizing Websockets and Bootstrap/JS

http://www.ympd.org

GNU General Public License v2.0

515 stars 143 forks source link

Make the Dirble API token configurable and use HTTPS #141

Closed lesderid closed 6 years ago

lesderid commented 7 years ago

Fixes #140

The API currently doesn't work in most browsers because Dirble doesn't have the required CORS header. I've sent them an e-mail about it. (This is not caused by this pull request, it's already broken.)

It also seems the default API token for Dirble has expired. It would be best to generate a new one and change it in the relevant places.

ghost commented 7 years ago

I applied this patch and I used a new Dirble API key in order to test. Unfortunately it doesn't work for me.

Is there any way to get some log regarding this function?

lesderid commented 7 years ago

@archphile Dirble still doesn't have the CORS headers, but it's coming soon according to the author.