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
Change default value for MQTT buffers max size #2288
Conversation
CC @simensrostad please have a look. |
@xnaveira could you please sign the CLA? |
I thought I had already done it, it is done now @carlescufi ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the commit message to follow https://docs.zephyrproject.org/latest/contribute/index.html#commit-guidelines, something like:
net: lib: aws_iot: Change default value for MQTT buffers max size
The current default value for AWS_IOT_MQTT_RX_TX_BUFFER_LEN and
AWS_IOT_MQTT_PAYLOAD_BUFFER_LEN which is 256 makes subscription
to shadow topics fail silently. Increasing to 1000 makes possible
to receive shadow updates.
Signed-off-by: Xavier Naveira <xnaveira@gmail.com>
The current default value for AWS_IOT_MQTT_RX_TX_BUFFER_LEN and AWS_IOT_MQTT_PAYLOAD_BUFFER_LEN which is 256 makes subscription to shadow topics fail silently. Increasing to 1000 makes possible to receive shadow updates. Signed-off-by: Xavier Naveira <xnaveira@gmail.com>
76e0823
to
440ae9b
Compare
Done! |
The current default value for AWS_IOT_MQTT_RX_TX_BUFFER_LEN and AWS_IOT_MQTT_PAYLOAD_BUFFER_LEN which is 256 makes subscription to shadow topics fail silently. Increasing to 1000 makes possible to receive shadow updates.
Similar issue: copercini/esp8266-aws_iot#7