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

fix for UTF8 strings that use more than 1 byte, check UTF8 specs #46

Closed Smiche closed 6 years ago

Smiche commented 6 years ago

UTF8 strings can be 1 to 4 8 bit bytes, example fix that covers broader scope of the UTF8 character spec. resolve #44

kum-deepak commented 6 years ago

Thanks! See if you can add a test case.

If not I will add before merging.

In the next version TextEncoder/TextDecoder would be used (though these are not supported in MS browsers yet).

Smiche commented 6 years ago

Alright, should be ready to merge.

kum-deepak commented 6 years ago

Many thanks, it seems ready to merge. I have made minor changes to the test case.

kum-deepak commented 6 years ago

Released as version 4.0.8 in NPM