Fixing WiFi code to allow HTTPS-GET instead of HTTP-GET (required for Arduino IoT Cloud OTA).

Author: aentinger

libraries/WiFi/src/WiFiHelpers.cpp

@@ -17,6 +17,7 @@
 #include "WiFi.h"
 #include "mbed.h"
 #include "utility/http_request.h"
+#include "utility/https_request.h"
 
 static FILE* target;
 
@@ -26,7 +27,13 @@ void body_callback(const char* data, uint32_t data_len) {
 
 int WiFiClass::download(char* url, const char* target_file) {
 	target = fopen(target_file, "wb");
-	HttpRequest* req = new HttpRequest(getNetwork(), HTTP_GET, url, &body_callback);
-	req->send(NULL, 0);
-	fclose(target);
+  HttpsRequest* req = new HttpsRequest(getNetwork(), nullptr, HTTP_GET, url, &body_callback);
+	if (req->send(NULL, 0) == NULL)
+  {
+    fclose(target);
+    return req->get_error();
+  }
+  int const size = ftell(target);
+  fclose(target);
+  return size;
 }

libraries/WiFi/src/utility/https_request.h

@@ -49,16 +49,20 @@ class HttpsRequest : public HttpRequestBase {
                  const char* ssl_ca_pem,
                  http_method method,
                  const char* url,
-                 Callback<void(const char *at, uint32_t length)> body_callback = 0)
+                 mbed::Callback<void(const char *at, uint32_t length)> body_callback = 0)
         : HttpRequestBase(NULL, body_callback)
     {
+        _error = 0;
+        _network = network;
+
         _parsed_url = new ParsedUrl(url);
         _request_builder = new HttpRequestBuilder(method, _parsed_url);
         _response = NULL;
 
         _socket = new TLSSocket();
         ((TLSSocket*)_socket)->open(network);
-        ((TLSSocket*)_socket)->set_root_ca_cert(ssl_ca_pem);
+        //((TLSSocket*)_socket)->set_root_ca_cert(ssl_ca_pem);
+        ((TLSSocket*)_socket)->set_root_ca_cert("/wlan/", 0);
         _we_created_socket = true;
     }
 
@@ -76,7 +80,7 @@ class HttpsRequest : public HttpRequestBase {
     HttpsRequest(TLSSocket* socket,
                  http_method method,
                  const char* url,
-                 Callback<void(const char *at, uint32_t length)> body_callback = 0)
+                 mbed::Callback<void(const char *at, uint32_t length)> body_callback = 0)
         : HttpRequestBase(socket, body_callback)
     {
         _parsed_url = new ParsedUrl(url);
@@ -91,7 +95,10 @@ class HttpsRequest : public HttpRequestBase {
 
 protected:
     virtual nsapi_error_t connect_socket(char *host, uint16_t port) {
-        return ((TLSSocket*)_socket)->connect(host, port);
+        SocketAddress socketAddress = SocketAddress();
+        socketAddress.set_port(port);
+        _network->gethostbyname(host, &socketAddress);
+        return ((TLSSocket*)_socket)->connect(socketAddress);
     }
 };