RockteMQ-AI commented on issue #10676:
URL: https://github.com/apache/rocketmq/issues/10676#issuecomment-5112810675
**Issue Evaluation**
Category: `type/bug` | Status: **Confirmed**
The full-TelemetryCommand logging in
`ClientActivity.processTelemetryException` has been verified against the
current `develop` branch.
**Root Cause:** `ClientActivity.java:362` — `log.warn("process client
telemetryCommand failed. request:{}", request, t)` serializes the complete
inbound `TelemetryCommand` protobuf when an internal processing error occurs.
**Impact:** Warning logs can become excessively large and may expose
client-side telemetry metadata beyond what is needed for diagnosis.
**Severity:** Low — diagnostics quality issue, no functional impact.
**Suggested fix:** Replace full `request` logging with a compact summary
(command type, status code, nonce, client type, pub/sub case) while preserving
debuggability.
An automated fix proposal will be generated. Reply `/approve` to proceed
with PR generation.
---
*Automated evaluation by github-manager-bot*
--
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]
