Abort update if http.begin() returns false. Fix a typo in httpUpdate.ino (espressif#2156)

Author: Jeroen88

libraries/HTTPUpdate/examples/httpUpdate/httpUpdate.ino

@@ -56,7 +56,7 @@ void loop(){
 
 switch (ret){
 case HTTP_UPDATE_FAILED:
- Serial.printf("HTTP_UPDATE_FAILD Error (%d): %s\n", httpUpdate.getLastError(), httpUpdate.getLastErrorString().c_str());
+ Serial.printf("HTTP_UPDATE_FAILED Error (%d): %s\n", httpUpdate.getLastError(), httpUpdate.getLastErrorString().c_str());
 break;
 
 case HTTP_UPDATE_NO_UPDATES:

libraries/HTTPUpdate/src/HTTPUpdate.cpp

@@ -49,22 +49,31 @@ HTTPUpdate::~HTTPUpdate(void)
 HTTPUpdateResult HTTPUpdate::update(WiFiClient& client, const String& url, const String& currentVersion)
 {
  HTTPClient http;
- http.begin(client, url);
+if(!http.begin(client, url))
+{
+return HTTP_UPDATE_FAILED;
+ }
 returnhandleUpdate(http, currentVersion, false);
 }
 
 HTTPUpdateResult HTTPUpdate::updateSpiffs(WiFiClient& client, const String& url, const String& currentVersion)
 {
  HTTPClient http;
- http.begin(client, url);
+if(!http.begin(client, url))
+{
+return HTTP_UPDATE_FAILED;
+ }
 returnhandleUpdate(http, currentVersion, true);
 }
 
 HTTPUpdateResult HTTPUpdate::update(WiFiClient& client, const String& host, uint16_t port, const String& uri,
 const String& currentVersion)
 {
  HTTPClient http;
- http.begin(client, host, port, uri);
+if(!http.begin(client, host, port, uri))
+{
+return HTTP_UPDATE_FAILED;
+ }
 returnhandleUpdate(http, currentVersion, false);
 }