search

RocketChat / Rocket.Chat.Go.SDK

Go SDK for REST API and Realtime api
MIT License
60 stars 58 forks source link

realtime: fix message decoding #15

Open yukithm opened 5 years ago

yukithm commented 5 years ago

update.args contains msg and options. Not multiple messages. args[0] is msg args[1] is options

See https://github.com/RocketChat/Rocket.Chat/blob/6c3c3790c65a4de87f6b36f219fe2d6b6aad3c94/server/stream/messages.js#L27

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

geekgonecrazy commented 5 years ago

Thanks for the contribution!

That's technically not the only place a message can be sent. We need to confirm no update event would ever contain more than one message.

I'm not confident that no place does currently.