Skip to content

Commit 24d7f8b

Browse files
Merge branch 'feature/reduce_mqtt5_ack_verbosity' into 'master'
feat(mqtt5): Reduce log verbosity for MQTT ACK Closes IDFGH-14616 See merge request espressif/esp-mqtt!230
2 parents 7e5e482 + 6b7b9c3 commit 24d7f8b

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

mqtt5_client.c

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ void esp_mqtt5_decrement_packet_counter(esp_mqtt5_client_handle_t client)
3636
void esp_mqtt5_parse_pubcomp(esp_mqtt5_client_handle_t client)
3737
{
3838
if (client->mqtt_state.connection.information.protocol_ver == MQTT_PROTOCOL_V_5) {
39-
ESP_LOGI(TAG, "MQTT_MSG_TYPE_PUBCOMP return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
39+
ESP_LOGD(TAG, "MQTT_MSG_TYPE_PUBCOMP return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
4040
size_t msg_data_len = client->mqtt_state.in_buffer_read_len;
4141
client->event.data = mqtt5_get_pubcomp_data(client->mqtt_state.in_buffer, &msg_data_len, &client->event.property->user_property);
4242
client->event.data_len = msg_data_len;
@@ -48,7 +48,7 @@ void esp_mqtt5_parse_pubcomp(esp_mqtt5_client_handle_t client)
4848
void esp_mqtt5_parse_puback(esp_mqtt5_client_handle_t client)
4949
{
5050
if (client->mqtt_state.connection.information.protocol_ver == MQTT_PROTOCOL_V_5) {
51-
ESP_LOGI(TAG, "MQTT_MSG_TYPE_PUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
51+
ESP_LOGD(TAG, "MQTT_MSG_TYPE_PUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
5252
size_t msg_data_len = client->mqtt_state.in_buffer_read_len;
5353
client->event.data = mqtt5_get_puback_data(client->mqtt_state.in_buffer, &msg_data_len, &client->event.property->user_property);
5454
client->event.data_len = msg_data_len;
@@ -60,7 +60,7 @@ void esp_mqtt5_parse_puback(esp_mqtt5_client_handle_t client)
6060
void esp_mqtt5_parse_unsuback(esp_mqtt5_client_handle_t client)
6161
{
6262
if (client->mqtt_state.connection.information.protocol_ver == MQTT_PROTOCOL_V_5) {
63-
ESP_LOGI(TAG, "MQTT_MSG_TYPE_UNSUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
63+
ESP_LOGD(TAG, "MQTT_MSG_TYPE_UNSUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
6464
size_t msg_data_len = client->mqtt_state.in_buffer_read_len;
6565
client->event.data = mqtt5_get_unsuback_data(client->mqtt_state.in_buffer, &msg_data_len, &client->event.property->user_property);
6666
client->event.data_len = msg_data_len;
@@ -72,7 +72,7 @@ void esp_mqtt5_parse_unsuback(esp_mqtt5_client_handle_t client)
7272
void esp_mqtt5_parse_suback(esp_mqtt5_client_handle_t client)
7373
{
7474
if (client->mqtt_state.connection.information.protocol_ver == MQTT_PROTOCOL_V_5) {
75-
ESP_LOGI(TAG, "MQTT_MSG_TYPE_SUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
75+
ESP_LOGD(TAG, "MQTT_MSG_TYPE_SUBACK return code is %d", mqtt5_msg_get_reason_code(client->mqtt_state.in_buffer, client->mqtt_state.in_buffer_read_len));
7676
}
7777
}
7878

@@ -113,7 +113,6 @@ esp_err_t esp_mqtt5_get_publish_data(esp_mqtt5_client_handle_t client, uint8_t *
113113

114114
if (property.topic_alias) {
115115
if (*msg_topic_len == 0) {
116-
ESP_LOGI(TAG, "Publish topic is empty, use topic alias");
117116
*msg_topic = esp_mqtt5_client_get_topic_alias(client->mqtt5_config->peer_topic_alias, property.topic_alias, msg_topic_len);
118117
if (!*msg_topic) {
119118
ESP_LOGE(TAG, "%s: esp_mqtt5_client_get_topic_alias() failed", __func__);

0 commit comments

Comments
 (0)