[GitHub] [rocketmq] humkum opened a new pull request, #6746: feat:add INNER_MULTI_DISPATCH check, can not contains '/' path separator

Fri, 12 May 2023 03:14:00 -0700 


humkum opened a new pull request, #6746:
URL: https://github.com/apache/rocketmq/pull/6746

   Fixes #issue_id
   
   ### Brief Description
   
   Check from client side and broker side:
   Client side: INNER_MULTI_DISPATCH contains "/" throws MQClientException to 
avoid illegal LMQ path
   Broker side: If receive old version clients message, generate CQ file for 
valid LMQ and skip generate CQ file for invalid LMQ
   
   ### How Did You Test This Change?
   Client exception:
   
![image](https://github.com/apache/rocketmq/assets/50660789/41b703ba-d91b-4f48-abc7-2f77baad2eb0)
   Broker log:
   
![image](https://github.com/apache/rocketmq/assets/50660789/a30611e5-440f-4fef-8afe-775466122d2e)
   
   To avoid too many logs with every message, the code had delete the error log 
when skip generate the invalid LMQ.
   
   


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

Reply via email to