Merge pull request #34 from tobiasschuerg/v1_ping_status_code_fix

tobiasschuerg · web-flow · commit 9dd77998ba25 · 2020-03-08T10:16:45.000+01:00
Fixes #33 wrong status code expected on version 1.x
diff --git a/src/InfluxDbClient.cpp b/src/InfluxDbClient.cpp
@@ -453,7 +453,8 @@ bool InfluxDBClient::validateConnection() {
         _lastErrorResponse = FPSTR(UnitialisedMessage);
         return false;
     }
-    String url = _serverUrl + (_dbVersion==2?"/ready":"/ping");
+    // on version 1.x /ping will by default return status code 204, without verbose
+    String url = _serverUrl + (_dbVersion==2?"/ready":"/ping?verbose=true");
     INFLUXDB_CLIENT_DEBUG("[D] Validating connection to %s\n", url.c_str());
 
     if(!_httpClient.begin(*_wifiClient, url)) {

Original file line number	Diff line number	Diff line change
`@@ -453,7 +453,8 @@ bool InfluxDBClient::validateConnection() {`
`453`	`453`	`_lastErrorResponse = FPSTR(UnitialisedMessage);`
`454`	`454`	`return false;`
`455`	`455`	`}`
`456`		`- String url = _serverUrl + (_dbVersion==2?"/ready":"/ping");`
	`456`	`+ // on version 1.x /ping will by default return status code 204, without verbose`
	`457`	`+ String url = _serverUrl + (_dbVersion==2?"/ready":"/ping?verbose=true");`
`457`	`458`	`INFLUXDB_CLIENT_DEBUG("[D] Validating connection to %s\n", url.c_str());`
`458`	`459`
`459`	`460`	`if(!_httpClient.begin(*_wifiClient, url)) {`