Skip to content

Commit

Permalink
Relax UPNP search target matching, don't depend on \r\n
Browse files Browse the repository at this point in the history
Fix discovery by Hue Essentials app, might be related to Echo as well.
  • Loading branch information
manup committed Oct 26, 2018
1 parent fb3abb9 commit 2c0c974
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions upnp.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,7 @@ void DeRestPluginPrivate::upnpReadyRead()
DBG_Printf(DBG_HTTP, "UPNP %s:%u\n%s\n", qPrintable(host.toString()), port, datagram.data());
datagram.clear();

if (searchTarget == "ST: ssdp:all\r\n" || searchTarget == "ST: upnp:rootdevice\r\n")
if (searchTarget.startsWith(QLatin1String("ST: ssdp:all")) || searchTarget.startsWith(QLatin1String("ST: upnp:rootdevice")))
{
datagram.append(response);
datagram.append(st1);
Expand All @@ -239,7 +239,7 @@ void DeRestPluginPrivate::upnpReadyRead()
}
datagram.clear();
}
if (searchTarget == "ST: ssdp:all\r\n" || searchTarget == QString(QLatin1String("ST: uuid:%1\r\n")).arg(gwConfig["uuid"].toString()))
if (searchTarget.startsWith(QLatin1String("ST: ssdp:all")) || searchTarget == QString(QLatin1String("ST: uuid:%1\r\n")).arg(gwConfig["uuid"].toString()))
{
datagram.append(response);
datagram.append(st2);
Expand All @@ -250,7 +250,7 @@ void DeRestPluginPrivate::upnpReadyRead()
}
datagram.clear();
}
if (searchTarget == "ST ssdp:all\r\n" || searchTarget == "ST: urn:schemas-upnp-org:device:basic:1\r\n")
if (searchTarget.startsWith(QLatin1String("ST: ssdp:all")) || searchTarget.startsWith(QLatin1String("ST: urn:schemas-upnp-org:device:basic:1")))
{
datagram.append(response);
datagram.append(st3);
Expand Down

0 comments on commit 2c0c974

Please sign in to comment.