Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
riwalker opened this issue Apr 9, 2024 · 3 comments
Open

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

riwalker opened this issue Apr 9, 2024 · 3 comments
Assignees
Labels
discussion This issues and pr are worth discussing help wanted Extra attention is needed wontfix This will not be worked on
Milestone

Comments

@riwalker
Copy link

riwalker commented Apr 9, 2024

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

  • OS: WIn11
  • MQTTX version 1.9.10
@riwalker riwalker added the bug Something isn't working label Apr 9, 2024
@ysfscream ysfscream self-assigned this Apr 9, 2024
@ysfscream ysfscream added the desktop MQTTX Desktop label Apr 9, 2024
@ysfscream ysfscream added this to the v1.10.0 milestone Apr 9, 2024
@ysfscream
Copy link
Member

ysfscream commented Apr 9, 2024

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
Copy link
Author

riwalker commented Apr 9, 2024 via email

@ysfscream
Copy link
Member

  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 ysfscream added help wanted Extra attention is needed discussion This issues and pr are worth discussing wontfix This will not be worked on and removed bug Something isn't working desktop MQTTX Desktop labels Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This issues and pr are worth discussing help wanted Extra attention is needed wontfix This will not be worked on
Projects
Status: Reject
Development

No branches or pull requests

2 participants