Send raw buffers using whisper and shout about zyre.js HOT 6 CLOSED

Maybe you can use the same api as the Readable Stream. So that you can change the encoding of messages using a method .setEncoding(encoding).

If you then want your new API to be backwards compatible with the current one, you could set the default encoding to "utf8". And users that need raw buffer access can simply call Zyre#setEncoding(null), an API they are already comfortable with since this is the common way to this in NodeJS API's.

from zyre.js.

Good idea! I will look into this and update the binary-messages branch.
Would be great if you test it when it's updated.

from zyre.js.

Yes, it's possible.
I am already testing it in binary-messages. I am just not sure about usability, because if I change the API, a console.log(msg) will result in a <Buffer 61 73 64 66>. You would have to use console.log(msg.toString()) to read string messages.
Sending messages would be no problem, as zyre.whisper() and zyre.shout() accept raw buffers as well.

If you want to test raw buffers now on your side, you could add "zyre.js": "interpretor/zyre.js#binary-messages" as a dependency in package.json.
Let me know how you think about it, and then we could add this in a new release.

from zyre.js.

Sure, I would be happy to test it!

And thanks for your nice API, it makes things way simpler than trying to use the official NodeJS Zyre binding 🙂.

from zyre.js.

I pushed a commit which adds support for multiple encodings plus the possibility to receive raw Buffers. d4fa967

from zyre.js.

Merged to master

from zyre.js.