wwbmmm opened a new pull request, #3099: URL: https://github.com/apache/brpc/pull/3099
### What problem does this PR solve? Issue Number: NULL Problem Summary: Add pb2json depth limit ### What is changed and the side effects? Changed: Add recursion depth limit for ProtoMessageToJson, ProtoMessageToProtoJson, JsonToProtoMessage, ProtoJsonToProtoMessage Side effects: - Performance effects: Slightly cost more CPU in the pb2json and json2pb functions - Breaking backward compatibility: pb message or json with a depth > 100 will convert fail. User can increase the gflag json2pb_max_recursion_depth to handle larger depth. --- ### Check List: - Please make sure your changes are compilable. - When providing us with a new feature, it is best to add related tests. - Please follow [Contributor Covenant Code of Conduct](https://github.com/apache/brpc/blob/master/CODE_OF_CONDUCT.md). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
