[patch #9813] MQTT: PINGRESP was sent too late

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

[patch #9813] MQTT: PINGRESP was sent too late

Wilfred
URL:
  <https://savannah.nongnu.org/patch/?9813>

                 Summary: MQTT: PINGRESP was sent too late
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: giusloq
            Submitted on: Tue 11 Jun 2019 07:40:15 AM UTC
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email:
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None

    _______________________________________________________

Details:

Attached is a patch that solves the bug #56022.

The first PINGRESP (keep-alive) was sent only after 2 *
MQTT_CYCLIC_TIMER_INTERVAL. If MQTT_CYCLIC_TIMER_INTERVAL = 5s and keep-alive
is set to 5s too, the first PINGRESP is sent 10s after the connection.

In this case, the server (MQTT Broker) could force a disconnection if the
PINGRESP message isn't received in 7-8s after connection.




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Tue 11 Jun 2019 07:40:15 AM UTC  Name: patch.diff  Size: 259B   By:
giusloq

<http://savannah.nongnu.org/patch/download.php?file_id=47071>

    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/patch/?9813>

_______________________________________________
  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
|

[patch #9813] MQTT: PINGRESP was sent too late

Wilfred
Update of patch #9813 (project lwip):

                Category:                    None => apps                  
                  Status:                    None => Done                  
             Assigned to:                    None => goldsimon              
             Open/Closed:                    Open => Closed                

    _______________________________________________________

Follow-up Comment #1:

Pushed, although slightly different.

    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/patch/?9813>

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


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