emqx / MQTTX

A Powerful and All-in-One MQTT 5.0 client toolbox for Desktop, CLI and WebSocket.
https://mqttx.app
Apache License 2.0
3.95k stars 450 forks source link

large TEXT as a publish starts to not obey a new line #1623

Closed riwalker closed 5 months ago

riwalker commented 7 months ago

What did I do

try and publish large data, using '/' as a new line esp_mqtt_client_publish(mqtt_client, "/menu/dmx", "1 - UDP - Send DMX frame\ 2 - UDP - Send DMX RAMP 4 colors\ 3 - UMAC - Send DMX frame\ 4 - UMAC - Send DMX RAMP 4 colors - 5B\ 5 - UMAC - Send DMX RAMP 4 colors - 400B\ 6 - UMAC - Send BLANK all LED\ 7 - Reset STATs\ 8 - Get STATs\ a - Set Default PLC Setup\ b - Set PLC Modulation type\ c - Set Min/Max Intra-Frame Time\ d - Discover Nodes\ n - Send message to node x\ u - Unicast - Multicast Address\ v - START Unicast sACN Transmit\ w - STOP unicast sACN Transmit\ x - START Multicast sACN Receive\ y - STOP Multicast sACN Receive", 0, 0, 0);

What happened

see here (it works for the first 6 lines, then 7 & 8 go crazy, then it works again !? image

Expected

all lines should align

Environment

ysfscream commented 7 months ago

Hi, thanks for reaching out. I've tested locally via CLI and the desktop app, and everything works well. Please take a look at the attached images for reference.

image image

Check the data format on your publishing side. To verify its accuracy, try printing or logging the data before publishing. Thanks for your understanding.

riwalker commented 7 months ago

Im sending as rawtext

The / symbol is supposed to be a new line

So in MQTT, any format here ?

I just moved the page separator, and it gets worse

From: Luke Yu @.> Sent: Monday, April 8, 2024 10:37 PM To: emqx/MQTTX @.> Cc: riwalker @.>; Author @.> Subject: Re: [emqx/MQTTX] large TEXT as a publish starts to not obey a new line (Issue #1623)

Hi, thanks for reaching out. I've tested locally via CLI and the desktop app, and everything works well. Please take a look at the attached images for reference.

image.png (view on web) https://github.com/emqx/MQTTX/assets/21299158/a9bc6cdb-9d74-4913-8d98-636b2f61b990 image.png (view on web) https://github.com/emqx/MQTTX/assets/21299158/a9bc6cdb-9d74-4913-8d98-636b2f61b990

Check the data format on your publishing side. To verify its accuracy, try printing or logging the data before publishing. Thanks for your understanding.

— Reply to this email directly, view it on GitHub https://github.com/emqx/MQTTX/issues/1623#issuecomment-2044049668 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIKT5WTJB7FJ26CIVVB22DY4NH5TAVCNFSM6AAAAABF5VIZ26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBUGA2DSNRWHA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/ABIKT5XXX34LYXFWUCY7QRTY4NH5TA5CNFSM6AAAAABF5VIZ26WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTZ2W4QI.gif Message ID: @. @.> >

ysfscream commented 7 months ago
  1. MQTT supports any data format, including UTF-8 text and binary data.
  2. Using "/" as a separator is not a standard practice.
  3. When MQTTX receives data and converts it to plaintext, it interprets it using UTF-8 encoding for standard text parsing.
ysfscream commented 5 months ago

If there is no update, I will close this issue; thanks for using it.