Documentation @ https://www.npmjs.com/package/stomp-client#stompaddress-port-user-pass-protocolversion-reconnectopts is slightly wrong

easternbloc / node-stomp-client

A STOMP client for Node.js

Other

93 stars 47 forks source link

Documentation @ https://www.npmjs.com/package/stomp-client#stompaddress-port-user-pass-protocolversion-reconnectopts is slightly wrong #40

Closed ArunMonySpin closed 9 years ago

ArunMonySpin commented 9 years ago

The documentation https://www.npmjs.com/package/stomp-client#stompaddress-port-user-pass-protocolversion-reconnectopts

mentions the "reconnectOptions" as the 6th argument, but in real its the 7th argument. I had to go thru the code to figure out this discrepancy as no obvious errors are thrown when this is offered as the 6th argument. kindly correct the documentation.

Slijkhuis commented 9 years ago

+1 this was quite confusing to me too. The sixth argument is 'vhost', which has a default of null.

easternbloc commented 9 years ago

Fixed in #44