Merge pull request #2038 from SmartThingsCommunity/hue_ssl_timeout

philips-hue: Re-set socket timeout on sockets after ssl wrap

Author: NoahCornell

drivers/SmartThings/philips-hue/src/lunchbox/sse/eventsource.lua

@@ -275,6 +275,10 @@ local function connecting_action(source)
         })
         if err ~= nil then return nil, err end
 
+        -- Re-set timeout due to cosock not carrying timeout over in some Lua library versions
+        err = select(2, source._sock:settimeout(60))
+        if err ~= nil then return nil, err end
+
         err = select(2, source._sock:dohandshake())
         if err ~= nil then return nil, err end
       end

drivers/SmartThings/philips-hue/src/utils/init.lua

@@ -460,6 +460,16 @@ function utils.labeled_socket_builder(label)
       if not sock or err ~= nil then
         return nil, (err and "SSL wrap error: " .. err) or "Unexpected nil socket returned from ssl.wrap"
       end
+      log.info(
+        string.format(
+          "%sSetting SSL socket timeout for Hue REST Connection", label
+        )
+      )
+      -- Re-set timeout due to cosock not carrying timeout over in some Lua library versions
+      err = select(2, sock:settimeout(60))
+      if err ~= nil then
+        return nil, "settimeout error: " .. err
+      end
       log.info(
         string.format(
           "%sPerforming SSL handshake for for Hue REST Connection", label