RockteMQ-AI commented on issue #10718:
URL: https://github.com/apache/rocketmq/issues/10718#issuecomment-5141740730
**Issue Evaluation**
Category: `type/bug` | Status: **Confirmed**
Verified against the current codebase on `develop` branch.
**Root Cause:** `GrpcClientSettingsManager` logs the full `Settings`
protobuf in the periodic cleanup path:
- Line ~285: `log.info("remove unused grpc client settings. group:{},
settings:{}", consumerGroupInfo, settings)`
This fires during the `doClean` cycle for stale client entries. The
`Settings` object can contain detailed publish/subscription metadata.
**Impact:** Periodic information leakage through info-level logs. Every
cleanup cycle dumps full Settings for each removed client.
**Severity:** medium — no functional impact, but cleaner logs should use a
compact summary instead of full protobuf output.
**Suggested Fix:** Replace `settings` in the log with a compact summary
(e.g., client type, publishing topic count, subscription count).
---
*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]
