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.

Sign up for GitHub

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

Jump to bottom

Blank MQTT payload crashes 8266 #185

Closed
xoseperez opened this issue Aug 15, 2017 · 3 comments
Closed

Blank MQTT payload crashes 8266 #185

xoseperez opened this issue Aug 15, 2017 · 3 comments
Labels
bug minor mqtt

Comments

@xoseperez
Copy link
Owner

Originally reported by: Wacko Madman (Bitbucket: wacko_madman, GitHub: Unknown)

Hello there, I've been using espurna for a couple of months now, and run into some problems when my mqtt broker publish on espurna's relay topic a blank message, causing a crash on the esp8266 (sonoff 1ch hardware)
@xoseperez
Copy link
Owner Author

Is there any reason to process a 0 length MQTT payload? If not, then easy way is to discard them...

@xoseperez
Copy link
Owner Author

Fixed with 1.9.0

@xoseperez
Copy link
Owner Author

Removing milestone: 1.8.4 (automated comment)

xoseperez added a commit that referenced this issue Jan 24, 2018
@xoseperez
Issue #185. Discard MQTT messages with payload size 0
2531bc6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug minor mqtt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xoseperez