gnodet-bot commented on code in PR #29736:
URL: https://github.com/apache/camel/pull/29736#discussion_r4553410510
##########
components/camel-whatsapp/src/main/java/org/apache/camel/component/whatsapp/service/WhatsAppServiceRestAPIAdapter.java:
##########
@@ -217,7 +217,8 @@ public void sendMessage(Exchange exchange, AsyncCallback
callback, T message, St
return;
}
int statusCode = response.statusCode();
- LOG.info("Response received with status {} and body {}",
statusCode, response.body());
+ LOG.info("Response received with status {}", statusCode);
+ LOG.debug("Response received with status {} and body {}",
statusCode, response.body());
Review Comment:
💡 **Minor nit:** When DEBUG logging is enabled, the status code appears
twice with the same prefix:
```
INFO Response received with status 200
DEBUG Response received with status 200 and body {...}
```
The split is valuable (body at DEBUG, not INFO) — but the duplicate prefix
makes the two lines feel redundant. Consider a distinct INFO message, e.g.:
```suggestion
LOG.info("Response received with status {}", statusCode);
LOG.debug("Response body: {}", response.body());
```
Non-blocking.
--
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]
