search

stomp-js / stomp-websocket

Stomp client for Web browsers and node.js apps
https://stomp-js.github.io/stomp-websocket/
Apache License 2.0
141 stars 36 forks source link

Support multi-byte string #43

Closed painnick closed 6 years ago

painnick commented 6 years ago

Modified stomp.coffee file to support multi-byte string. And add unit-test code.

kum-deepak commented 6 years ago

Many thanks, I will review your changes.

In addition I will also port the changes to https://github.com/stomp-js/stompjs - next version of this library. You can follow the progress at https://github.com/stomp-js/stompjs/issues/2

kum-deepak commented 6 years ago

One interesting observation. The test case passes with the old code as well. Please check.

Smiche commented 6 years ago

Alternative PR that has the same result issue #46

kum-deepak commented 6 years ago

Closing, in favor of #46

kum-deepak commented 6 years ago

Released as version 4.0.8 in NPM