notandy / ympd

Standalone MPD Web GUI written in C, utilizing Websockets and Bootstrap/JS
http://www.ympd.org
GNU General Public License v2.0
523 stars 144 forks source link

Change websocket URL to allow easy incorporation of different backends. #40

Closed iwanders closed 9 years ago

iwanders commented 10 years ago

I propose a slight change to the javascript function which creates the websocket URL.

This way the websocket connects on an URL different from the webpage. This makes it easier to use a different backend, for example the Python backend I created (https://github.com/iwanders/ympd_python).

The original C backend does not require any changes to continue working.

notandy commented 10 years ago

Hi,

could you please squash the two commits? ( https://help.github.com/articles/about-git-rebase/ ) I still need to test the commit since older versions of mongoose websocket server had issues with named websocket connections, but I'am confident that's already fixed.

iwanders commented 10 years ago

Done. Thanks for the pointer, I did not know this was possible.

laclaro commented 10 years ago

Is this also the reason for https://github.com/notandy/ympd/issues/24 and problems running ympd behind a proxy?