SmartThingsCommunity / SmartThingsEdgeDrivers

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Bose driver discovery: possible bug

toddaustin07 opened this issue · comments

Line 99 of discovery.lua

elseif err == "timeout" and socket ~= nil then

Should reference to 'socket' be 'sock'? Even so, not sure it still makes sense...

Hi @toddaustin07,

You are correct that this should be sock instead of socket. I went ahead and fixed it here. In this case, if sock ~= nil then we have already sent the ssdp discovery request and since the error is timeout we break out of the loop as we have likely received all the responses to the initial request.