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

Hotfix/handle frames length #31

Closed pawanaraballi closed 5 years ago

pawanaraballi commented 6 years ago

I am handling a case when there the frames value is one.

kum-deepak commented 6 years ago

Thanks @pawanaraballi for the PR. Please explain in which case the original code was failing. Adding server details etc. will be helpful.

The js file that you have edited is a generated file. The original source is in coffee script. Please check https://stomp-js.github.io/stomp-websocket/codo/extra/docs-src/Contribute.md.html

kum-deepak commented 6 years ago

Also add a test case that fails without the fix and succeeds with the fix. Many thanks!

Let me know if you need any assistance.

kum-deepak commented 6 years ago

Planning for next version of stompjs is on the way. Your participation will be really appreciated, please join at https://github.com/stomp-js/stompjs/issues/1