Skip to content

Commit

Permalink
chore(aclk/mqtt): remove client_id len check (netdata#19247)
Browse files Browse the repository at this point in the history
chore(aclk/mqtt):remove client_id len check
  • Loading branch information
ilyam8 authored Dec 18, 2024
1 parent c3b8eef commit 6128e50
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/aclk/mqtt_websockets/mqtt_ng.c
Original file line number Diff line number Diff line change
Expand Up @@ -823,12 +823,6 @@ mqtt_msg_data mqtt_ng_generate_connect(struct transaction_buffer *trx_buf,
// at his own risk!
nd_log(NDLS_DAEMON, NDLP_WARNING, "client_id provided is empty string. This might not be allowed by server [MQTT-3.1.3-6]");
}
if(len > MQTT_MAX_CLIENT_ID) {
// [MQTT-3.1.3-5] server MUST allow client_id length 1-32
// server MAY allow longer client_id, if user provides longer client_id
// warn them he is doing so at his own risk!
nd_log(NDLS_DAEMON, NDLP_WARNING, "client_id provided is longer than 23 bytes, server might not allow that [MQTT-3.1.3-5]");
}

if (lwt) {
if (lwt->will_message && lwt->will_message_size > 65535) {
Expand Down

0 comments on commit 6128e50

Please sign in to comment.