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

   ### Which Issue(s) This PR Fixes
   
   Fixes #10795
   
   ### Brief Description
   
   - Validate Proxy remoting `CHECK_CLIENT_CONFIG` request bodies instead of 
always returning success
   - Keep TAG expressions accepted
   - Reject SQL/property filters when `enablePropertyFilter` is disabled, 
matching the default broker behavior
   - Compile SQL/property filter expressions when enabled and return 
`SUBSCRIPTION_PARSE_FAILED` on parse errors
   - Add regression coverage for TAG, disabled property filter, and invalid SQL 
expression cases
   
   ### How Did You Test This Change?
   
   - `mvn -pl proxy -Dtest=ClientManagerActivityTest test`
   
   The target test passed: 3 tests, 0 failures, 0 errors. The Maven run 
completed with BUILD SUCCESS; the output still includes existing Jacoco/JDK17 
instrumentation warnings.


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