[bug #54655] MQTT restricts password to be less than 256 bytes

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[bug #54655] MQTT restricts password to be less than 256 bytes

Simon Goldschmidt
URL:
  <https://savannah.nongnu.org/bugs/?54655>

                 Summary: MQTT restricts password to be less than 256 bytes
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: richmond_umagat
            Submitted on: Thu 13 Sep 2018 07:27:21 AM UTC
                Category: apps
                Severity: 3 - Normal
              Item Group: Feature Request
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None
            lwIP version: git head

    _______________________________________________________

Details:

MQTT brokers such as Google Cloud IoT Core requires MQTT clients to send JSON
Web Token (JWT) as password field of the MQTT Connect packet. JWT can be more
than 255 bytes.

Currently, the MQTT library restricts password to be less than 256 bytes, thus
it prevents connectivity to Google Cloud IoT Core.

To fix the issue, mqtt_client_connect() must be updated as follows:
- Change client_pass_len from u8 to u16
- Update error checking length of client_info->client_pass from 255 to X
- Also, MQTT_OUTPUT_RINGBUF_SIZE has been changed from 256 to 768.
Modifications are enclosed in FT32_PORT.

Attached are the proposed modifications tested to connect to Google Cloud IoT
Core.




    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?54655>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #54655] MQTT restricts password to be less than 256 bytes

Simon Goldschmidt
Additional Item Attachment, bug #54655 (project lwip):

File name: mqtt.c                         Size:49 KB
File name: mqtt_opts.h                    Size:3 KB


    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?54655>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #54655] MQTT restricts password to be less than 256 bytes

Simon Goldschmidt
Update of bug #54655 (project lwip):

                  Status:                    None => Fixed                  
             Assigned to:                    None => goldsimon              
             Open/Closed:                    Open => Closed                
         Planned Release:                    None => 2.1.0                  

    _______________________________________________________

Follow-up Comment #1:

Fixed, thanks for reporting.

    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?54655>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel