Aias00 opened a new pull request, #10677:
URL: https://github.com/apache/rocketmq/pull/10677
### Summary
- Replace full inbound `TelemetryCommand` logging in
`ClientActivity.processTelemetryException` with a compact request summary.
- Preserve diagnostic fields such as command type, status code, nonce,
client type, and pub/sub case.
- Add coverage to ensure thread stack trace telemetry content is omitted
from summaries.
### Motivation
Closes #10676.
When client telemetry processing fails with an internal error, the previous
warning log serialized the full protobuf request via `request:{}`. Client
telemetry may include verbose or sensitive runtime details such as thread stack
traces, so logs should stay bounded and diagnostic-focused.
### Tests
- `mvn -pl proxy -Dtest=ClientActivityTest -DfailIfNoTests=false test`
Result: BUILD SUCCESS; 16 tests passed; checkstyle reported 0 violations.
The run emits existing JaCoCo instrumentation warnings under the local JDK, but
tests and build completed successfully.
--
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]
