Socket .connect function is hard coded to connect to ws://localhost:8000/socketcluster/

sacOO7 / socketcluster-client-java

Native java and android client for socketcluster framework in node.js

http://socketcluster.io/

Apache License 2.0

94 stars 43 forks source link

Socket .connect function is hard coded to connect to ws://localhost:8000/socketcluster/ #1

Closed heiba closed 7 years ago

heiba commented 7 years ago

Hello,

In your Socket.class, line 347, you have hard coded the class to connect to ws://localhost:8000/socketcluster/ and you're totally ignoring the url which is input in the Socket constructor new Socket(url).

public void connect() { try { this.ws = this.factory.createSocket("ws://localhost:8000/socketcluster/"); } catch (IOException var10) { System.out.printf(var10.getMessage(), new Object[0]); }

Please fix

sacOO7 commented 7 years ago

Hi , I have removed hard-coded URL from Socket.class. You can now download latest dependency jar. Sorry for inconvenience caused. More issues are always welcome :)