Merge pull request #3 from bringauto/BAF-841/fix_reconnect

MarioIvancik · web-flow · commit 2c8eacefca23 · 2024-05-17T09:17:26.000+02:00
BAF-841 / Fix reconnect sequence
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -7,7 +7,7 @@ SET(CMAKE_CXX_STANDARD 20)
 SET(CMAKE_CXX_EXTENSIONS OFF)   # For increased portability
 SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
-SET(INTERNAL_CLIENT_VERSION 1.1.0)
+SET(INTERNAL_CLIENT_VERSION 1.1.1)
 
 OPTION(BRINGAUTO_SAMPLES OFF)
 OPTION(BRINGAUTO_TESTS "Enable tests" OFF)
diff --git a/source/Communication.cpp b/source/Communication.cpp
@@ -31,7 +31,7 @@ int Communication::startReconnectSequence(Context *context, struct buffer status
 	if (Communication::sendConnectMessage(context) != OK) {
 		return UNABLE_TO_CONNECT;
 	}
-	if (Communication::readConnectResponse(context) == OK) {
+	if (Communication::readConnectResponse(context) != OK) {
 		return UNABLE_TO_CONNECT;
 	}
 	if(context->sendMessage(statusMessage) <= 0) {

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ int Communication::startReconnectSequence(Context *context, struct buffer status`
`31`	`31`	`if (Communication::sendConnectMessage(context) != OK) {`
`32`	`32`	`return UNABLE_TO_CONNECT;`
`33`	`33`	`}`
`34`		`- if (Communication::readConnectResponse(context) == OK) {`
	`34`	`+ if (Communication::readConnectResponse(context) != OK) {`
`35`	`35`	`return UNABLE_TO_CONNECT;`
`36`	`36`	`}`
`37`	`37`	`if(context->sendMessage(statusMessage) <= 0) {`