You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2021/12/11 01:52:40 UTC

[GitHub] [rocketmq] yuz10 commented on pull request #3386: [ISSUE #3369]Feature/enhancement fix illegal json by fastjson lx

yuz10 commented on pull request #3386:
URL: https://github.com/apache/rocketmq/pull/3386#issuecomment-991408915


   > > The change is not compatible for go and cpp client, So the best way is add a new version of protocol.
   > 
   > The go and cpp client both use the standard json serializer and deserializer libraries. In fact, the go and cpp client have not implemented those complex invalid json-formatted requests and responses, so they have not faced this problem yet, but if they continue introducing new features, they will also be stuck in this problem at one day.
   
   I dont think so, The line https://github.com/apache/rocketmq-client-go/blob/master/internal/route.go#L533 is not compatible with standard json.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@rocketmq.apache.org

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