astarte-platform / astarte_vmq_plugin

Astarte VerneMQ plugin publishes incoming messages on an AMQP exchange with some additional metadata, so they can be processed by other components.
http://astarte-platform.org/
Apache License 2.0
16 stars 10 forks source link

Add a patch to increase SSL handshake timeout in VerneMQ #62

Closed rbino closed 2 years ago

rbino commented 2 years ago

This is a temporary workaround to allow constrained devices that take more than 5 seconds to perform the SSL handshake to connect to the broker without timing out.

See https://github.com/vernemq/vernemq/issues/1670 for the related issue, this will be reverted once the configuration for the handshake timeout is exposed in VerneMQ

Signed-off-by: Riccardo Binetti riccardo.binetti@secomind.com

codecov[bot] commented 2 years ago

Codecov Report

Merging #62 (68e52e8) into release-1.0 (16cf4b1) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 68e52e8 differs from pull request most recent head e343838. Consider uploading reports for the commit e343838 to get more accurate results

@@             Coverage Diff              @@
##           release-1.0      #62   +/-   ##
============================================
  Coverage        73.18%   73.18%           
============================================
  Files                9        9           
  Lines              179      179           
============================================
  Hits               131      131           
  Misses              48       48           

Help us with your feedback. Take ten seconds to tell us how you rate us.

varnerac commented 2 years ago

Sorry, I thought this was upstream at VMQ. I didn't mean to leave this review in another repo. Terribly sorry. I have not had my coffee yet.