ETM-Schurig/powersync-app
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix misinterpretation of notifications with id 0

Browse Source 
might happen for hte very first connection after a nymead startup
This commit is contained in:
Michael Zanetti 2018-06-18 13:10:15 +02:00
parent 23800e9f97
commit 377766b043
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libmea-core/jsonrpc/jsonrpcclient.cpp
View File

@ -269,7 +269,7 @@ void JsonRpcClient::dataReceived(const QByteArray &data)
// Check if this is the initial handshake
if (dataMap.value("id").toInt() == 0 && dataMap.contains("params")) {
if (dataMap.value("id").toInt() == 0 && dataMap.contains("params") && !dataMap.contains("notification")) {
dataMap = dataMap.value("params").toMap();
m_initialSetupRequired = dataMap.value("initialSetupRequired").toBool();
m_authenticationRequired = dataMap.value("authenticationRequired").toBool();
@ -286,6 +286,7 @@ void JsonRpcClient::dataReceived(const QByteArray &data)
QVersionNumber minimumRequiredVersion = QVersionNumber(1, 0);
QVersionNumber protocolVersion = QVersionNumber::fromString(protoVersionString);
if (protocolVersion < minimumRequiredVersion) {
qWarning() << "Nymea box doesn't support minimum required version. Required:" << minimumRequiredVersion << "Found:" << protocolVersion;
m_connection->disconnect();
emit invalidProtocolVersion(protocolVersion.toString(), minimumRequiredVersion.toString());
return;