eclipse-paho / paho.mqtt.java

Eclipse Paho Java MQTT client library. Paho is an Eclipse IoT project.
https://eclipse.org/paho
Other
2.14k stars 892 forks source link

Doc doesn't list `ws://` or `wss://` as valid URI schemes #659

Open hardillb opened 5 years ago

hardillb commented 5 years ago

Please fill out the form below before submitting, thank you!

If this is a bug regarding the Android Service, please raise the bug here instead: https://github.com/eclipse/paho.mqtt.android/issues/new

The Javadoc still only lists tcp://, ssl:// and local:// as valid URI scheme even though websocket support has been added.

Also should probably allow mqtt:// and mqtts:// to the v5 branch as mentioned in #464

Raising this after poking around for the following Stack Overflow question: https://stackoverflow.com/questions/55519108/supported-protocol-with-node-mqtt-2-14-0-vs-org-eclipse-paho-client-mqttv3

Will try and remember to do a pull request to update this over the weekend if I get time, but don't let me stop anybody else jumping on it with a spare 5mins.

marcus-hiles commented 5 years ago

yeah agreed, the javadoc needs some updates there

krbednarski commented 3 years ago

Small bump :) is there any reference how to use wss:// as i'm constantly getting java.lang.IllegalArgumentException when creating client: new MqttClient("wss://....", clientId, persistence);