[GitHub] [rocketmq-client-cpp] ShannonDing opened a new issue #306: the binary message body should be added to the signatures.

[GitBox <gi...@apache.org>]

ShannonDing opened a new issue #306:
URL: https://github.com/apache/rocketmq-client-cpp/issues/306


   in the src/common/ClientRPCHook.cpp,
   ```const MemoryBlock* pBody = request.GetBody();
     if (pBody && pBody->getSize() > 0) {
       const char* msg_body = const_cast<const char*>(static_cast<char*>(pBody->getData()));
       LOG_DEBUG("msgBody is:%s, msgBody length is:%d", msg_body, pBody->getSize());
       totalMsg.append(msg_body, pBody->getSize());
     } else if (request.getMsgBody().length() > 0) {
       LOG_DEBUG("msgBody is:%s, msgBody length is:" SIZET_FMT "", request.getMsgBody().c_str(),
                 request.getMsgBody().length());
       totalMsg.append(request.getMsgBody());
     }
   ```
   We should add the message body to signature as default.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org