DOCUP - pubsubclient README.md

Moderators: grovkillen, Stuntteam, TD-er

Post Reply
Message
Author
dampa
Normal user
Posts: 87
Joined: 19 Jul 2018, 01:48

DOCUP - pubsubclient README.md

#1 Post by dampa » 24 Aug 2019, 12:49

I happened to trad the README.md and saw this line
The maximum message size, including header, is **128 bytes** by default. This is configurable via `MQTT_MAX_PACKET_SIZE` in `PubSubClient.h`.
but I looked at one of the messages I was sending it was 269 bytes long (??) so I went and looked into PubSubClient.h and it looks to me that the default is now 1024 bytes,
// MQTT_MAX_PACKET_SIZE : Maximum packet size
#ifndef MQTT_MAX_PACKET_SIZE
#define MQTT_MAX_PACKET_SIZE 1024 // need to fix this here, because this define cannot be overruled within the Arduino sketch...
#endif
Just thought you would want to update the README.md
Last edited by dampa on 24 Aug 2019, 19:58, edited 1 time in total.

User avatar
grovkillen
Core team member
Posts: 3621
Joined: 19 Jan 2017, 12:56
Location: Hudiksvall, Sweden
Contact:

Re: DOCUP - pubsubclient README.md

#2 Post by grovkillen » 24 Aug 2019, 13:05

Bits vs Bytes
ESP Easy Flasher [flash tool and wifi setup at flash time]
ESP Easy Webdumper [easy screendumping of your units]
ESP Easy Netscan [find units]
Official shop: https://firstbyte.shop/
Sponsor ESP Easy, we need you :idea: :idea: :idea:

User avatar
grovkillen
Core team member
Posts: 3621
Joined: 19 Jan 2017, 12:56
Location: Hudiksvall, Sweden
Contact:

Re: DOCUP - pubsubclient README.md

#3 Post by grovkillen » 24 Aug 2019, 13:06

Sorry see now...
ESP Easy Flasher [flash tool and wifi setup at flash time]
ESP Easy Webdumper [easy screendumping of your units]
ESP Easy Netscan [find units]
Official shop: https://firstbyte.shop/
Sponsor ESP Easy, we need you :idea: :idea: :idea:

TD-er
Core team member
Posts: 8643
Joined: 01 Sep 2017, 22:13
Location: the Netherlands
Contact:

Re: DOCUP - pubsubclient README.md

#4 Post by TD-er » 24 Aug 2019, 23:41

grovkillen wrote: 24 Aug 2019, 13:05 Bits vs Bytes
Nope, the size of the MQTT buffer has been increased a while ago.
And last week I merged a commit which actually allows to use the bigger buffer with messages over 384 bytes (which was for a long time the max. size)

Post Reply

Who is online

Users browsing this forum: Ahrefs [Bot] and 18 guests