fix: setting httpReadTimeout as connect timeout for HTTP connection on ESP32 (#149)

vlastahajek · vlastahajek · commit 2e7fe10c48c3 · 2021-07-09T21:54:28.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,8 @@
 # Changelog
+## 3.8.1 [in progress]
+### Fixes
+ - [#150](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/150) - `HTTPOptions::httpReadTimeout` is also set as the connect timeout for HTTP connection on ESP32. It doesn't work for HTTPS connection yet. 
+
 ## 3.8.0 [2021-04-01]
 ### Features
  - [#143](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/143) - `InfluxDBClient::setInsecure` now works also for ESP32. Requires Arduino ESP32 SDK 1.0.5 or higher
diff --git a/src/InfluxDbClient.cpp b/src/InfluxDbClient.cpp
@@ -319,6 +319,9 @@ void InfluxDBClient::setHTTPOptions(const HTTPOptions & httpOptions) {
     }
     _httpClient->setReuse(_httpOptions._connectionReuse);
     _httpClient->setTimeout(_httpOptions._httpReadTimeout);
+#if defined(ESP32) 
+     _httpClient->setConnectTimeout(_httpOptions._httpReadTimeout);
+#endif
 }
 
 void InfluxDBClient::resetBuffer() {

Original file line number	Diff line number	Diff line change
`@@ -319,6 +319,9 @@ void InfluxDBClient::setHTTPOptions(const HTTPOptions & httpOptions) {`
`319`	`319`	`}`
`320`	`320`	`_httpClient->setReuse(_httpOptions._connectionReuse);`
`321`	`321`	`_httpClient->setTimeout(_httpOptions._httpReadTimeout);`
	`322`	`+#if defined(ESP32)`
	`323`	`+ _httpClient->setConnectTimeout(_httpOptions._httpReadTimeout);`
	`324`	`+#endif`
`322`	`325`	`}`
`323`	`326`
`324`	`327`	`void InfluxDBClient::resetBuffer() {`