twei43846-afk opened a new pull request, #10742:
URL: https://github.com/apache/rocketmq/pull/10742

   ### Summary
   
   - Return a clearer `UNRECOGNIZED_CLIENT_TYPE` status message when Proxy gRPC 
heartbeat rejects an unsupported client type.
   - Return the same diagnostic from client termination when cached settings 
contain an unsupported client type.
   - Add focused `ClientActivityTest` coverage for both paths.
   
   ### Motivation
   
   Fixes #10671.
   
   The previous response only echoed the enum name, for example 
`CLIENT_TYPE_UNSPECIFIED`, which does not explain why the Proxy rejected the 
client or which client types are accepted. This keeps the behavior compatible 
while making the rejection reason visible to clients and adding a Proxy-side 
warning log.
   
   ### Tests
   
   - `git diff --check`
   - `mvn -Dmaven.repo.local=$env:USERPROFILE\.m2\repository -pl proxy -am 
-Dtest=ClientActivityTest -DfailIfNoTests=false test`
   
   Result: `ClientActivityTest` passed with 17 tests. The local JDK17 run 
emitted existing JaCoCo 0.8.5 instrumentation warnings for class file version 
61, but Maven exited successfully and surefire reports 0 failures / 0 errors.


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