search

SolaceProducts / pubsubplus-go-client

A Solace PubSub+ Go Client
Apache License 2.0
6 stars 8 forks source link

Setting ApplicationMessageID Overwrites ApplicationMessageType #5

Closed kenanelgaouny closed 2 years ago

kenanelgaouny commented 2 years ago

Hello 👋

it looks like the following methods are not behaving as expected: https://github.com/SolaceProducts/pubsubplus-go-client/blob/acba067eefbadce3c4896baa879d0acde8d47590/internal/ccsmp/ccsmp_message.go#L245-L261

The ApplicationMessageType is being set/retrieved instead of the applicationMessageID

i.e. solClient_msg_setApplicationMsgType and solClient_msg_getApplicationMsgType

instead of: solClient_msg_setApplicationMessageId and solClient_msg_getApplicationMessageId

mvsolace commented 2 years ago

Hi Kenan,

Thank you for the raising this issue. We will be addressing it with the next release of the API!

Mrc0113 commented 2 years ago

Thanks for opening this issue @kenanelgaouny. We appreciate you helping us improve the Go client! I just wanted to let you know that we have a developer community forum over at https://solace.community if you have any questions or even want to share what you create. Basically any sort of conversation you wouldn't usually have via a github issue ;)

Thanks again