ESP8266 authentication

README.md

@@ -52,6 +52,7 @@ A special thanks go to these people who helped me making this library
 + [x] ArduinoJSON 6 support
 
 ### Changelog
++ 2.1.14 ESP8266 authentication issue when user defined WiFi connection 
 + 2.1.13 ESP32 compile error
 + 2.1.12 No more fingerprint certificate validation for ESP8266
 + 2.1.11 Fixed an issue on group/chat ID in callback queries

library.json

@@ -7,7 +7,7 @@
     "type": "git",
     "url": "https://github.com/shurillu/CTBot.git"
   },
-  "version": "2.1.13",
+  "version": "2.1.14",
   "authors": {
     "name": "Stefano Ledda",
     "email": "[email protected]"

library.properties

@@ -1,5 +1,5 @@
 name=CTBot
-version=2.1.13
+version=2.1.14
 author=Stefano Ledda <[email protected]>
 maintainer=Stefano Ledda <[email protected]>
 sentence=Simple Arduino Telegram BOT library for ESP8266/ESP32

src/CTBotSecureConnection.cpp

@@ -9,6 +9,10 @@
 CTBotSecureConnection::CTBotSecureConnection() {
 	m_useDNS = true;
 #if defined(ARDUINO_ARCH_ESP8266) 
+	// moved from CTBotWiFiSetup::wifiConnect() to here. Time must be synch here because wifiSetup could be not used 
+	// (user defined wifi connection i.e. WiFiManager)
+	configTime(3 * 3600, 0, "pool.ntp.org", "time.nist.gov");
+	m_timeSynch = false;
 	m_cert.append(m_CAcert);
 #endif
 }
@@ -52,6 +56,18 @@ String CTBotSecureConnection::send(const String& message) {
 #elif defined(ARDUINO_ARCH_ESP8266) && CTBOT_USE_FINGERPRINT == 1 // ESP8266 with HTTPS verification
 	BearSSL::WiFiClientSecure telegramServer;
 //	telegramServer.setFingerprint(m_fingerprint);
+
+	// moved from CTBotWiFiSetup::wifiConnect() to here. Time must be synch here because wifiSetup could be not used 
+	// (user defined wifi connection i.e. WiFiManager)
+	if (!m_timeSynch) {
+		time_t now = time(nullptr);
+		if (now < 8 * 3600 * 2) {
+			serialLog("\n--->connect: Unable to sync time data.\n", CTBOT_DEBUG_WIFI);
+		}
+		else
+			m_timeSynch = true;
+	}
+
 	telegramServer.setTrustAnchors(&m_cert);
 	serialLog(FSTR("ESP8266 with https verification"), CTBOT_DEBUG_CONNECTION);
 #elif defined(ARDUINO_ARCH_ESP32) // ESP32
@@ -79,7 +95,7 @@ String CTBotSecureConnection::send(const String& message) {
 			IPAddress telegramServerIP;
 			telegramServerIP.fromString(TELEGRAM_IP);
 			if (!telegramServer.connect(telegramServerIP, TELEGRAM_PORT)) {
-				serialLog(FSTR("\nUnable to connect to Telegram server\n"), CTBOT_DEBUG_CONNECTION);
+				serialLog(FSTR("\nUnable to connect to Telegram server (URL)\n"), CTBOT_DEBUG_CONNECTION);
 				return("");
 			}
 			else {
@@ -96,7 +112,7 @@ String CTBotSecureConnection::send(const String& message) {
 		IPAddress telegramServerIP; // (149, 154, 167, 220);
 		telegramServerIP.fromString(TELEGRAM_IP);
 		if (!telegramServer.connect(telegramServerIP, TELEGRAM_PORT)) {
-			serialLog(FSTR("\nUnable to connect to Telegram server\n"), CTBOT_DEBUG_CONNECTION);
+			serialLog(FSTR("\nUnable to connect to Telegram server (IP)\n"), CTBOT_DEBUG_CONNECTION);
 			return("");
 		}
 		else

src/CTBotSecureConnection.h

@@ -44,6 +44,7 @@ class CTBotSecureConnection
 private:
 	bool            m_useDNS;
 	CTBotStatusPin  m_statusPin;
+	bool            m_timeSynch;
 
 #if defined(ARDUINO_ARCH_ESP8266)
 	X509List        m_cert;

src/CTBotWifiSetup.cpp

@@ -114,6 +114,9 @@ bool CTBotWifiSetup::wifiConnect(const String& ssid, const String& password) {
 
 	if (WiFi.status() == WL_CONNECTED) {
 
+// moved in CTBotSecureConnection::Send() + constructor because the user can manage by himself the wifi connection
+		// i.e. WifiManager
+/*
 #if defined(ARDUINO_ARCH_ESP8266) 
 		// Set time via NTP, as required for x.509 validation
 		configTime(3 * 3600, 0, "pool.ntp.org", "time.nist.gov");
@@ -144,6 +147,8 @@ bool CTBotWifiSetup::wifiConnect(const String& ssid, const String& password) {
 		serialLog(asctime(&timeinfo), CTBOT_DEBUG_WIFI);
 		serialLog("\n", CTBOT_DEBUG_WIFI);
 #endif
+*/
+
 
 		IPAddress ip = WiFi.localIP();
 		message = (String)FSTR("\nWiFi connected\nIP address: ") + ip.toString() + (String)"\n";