IPCore deprecated string-based API removal

String-based functions should be replaced with an explicit DNS hostname resolution and a SocketAddress-based API.

Author: michalpasztamobica

UNITTESTS/stubs/AT_CellularContext_stub.cpp

@@ -116,11 +116,6 @@ nsapi_error_t AT_CellularContext::get_ip_address(SocketAddress *address)
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-const char *AT_CellularContext::get_ip_address()
-{
-    return NULL;
-}
-
 void AT_CellularContext::attach(Callback<void(nsapi_event_t, intptr_t)> status_cb)
 {
 }

UNITTESTS/stubs/NetworkInterface_stub.cpp

@@ -31,10 +31,6 @@ nsapi_error_t NetworkInterface::get_ip_address(SocketAddress *)
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-const char *NetworkInterface::get_ip_address()
-{
-    return nullptr;
-}
 nsapi_error_t NetworkInterface::get_ipv6_link_local_address(SocketAddress *address)
 {
     return NSAPI_ERROR_UNSUPPORTED;
@@ -45,31 +41,16 @@ nsapi_error_t NetworkInterface::get_netmask(SocketAddress *)
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-const char *NetworkInterface::get_netmask()
-{
-    return nullptr;
-}
-
 nsapi_error_t NetworkInterface::get_gateway(SocketAddress *)
 {
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-const char *NetworkInterface::get_gateway()
-{
-    return nullptr;
-}
-
 nsapi_error_t NetworkInterface::set_network(const SocketAddress &ip_address, const SocketAddress &netmask, const SocketAddress &gateway)
 {
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-nsapi_error_t NetworkInterface::set_network(const char *ip_address, const char *netmask, const char *gateway)
-{
-    return NSAPI_ERROR_UNSUPPORTED;
-}
-
 nsapi_error_t NetworkInterface::set_dhcp(bool dhcp)
 {
     return NSAPI_ERROR_UNSUPPORTED;

UNITTESTS/stubs/NetworkStack_stub.cpp

@@ -104,11 +104,6 @@ nsapi_error_t NetworkStack::get_ip_address(SocketAddress* address)
     return NSAPI_ERROR_UNSUPPORTED;
 }
 
-const char *NetworkStack::get_ip_address()
-{
-    return NULL;
-}
-
 nsapi_error_t NetworkStack::get_ipv6_link_local_address(SocketAddress *address)
 {
     return NSAPI_ERROR_UNSUPPORTED;
@@ -118,8 +113,3 @@ nsapi_error_t NetworkStack::get_ip_address_if(SocketAddress* address, const char
 {
     return NSAPI_ERROR_UNSUPPORTED;
 }
-
-const char *NetworkStack::get_ip_address_if(const char *interface_name)
-{
-    return NULL;
-}

UNITTESTS/stubs/NetworkStack_stub.h

@@ -40,10 +40,6 @@ class NetworkStackstub : public NetworkStack {
     {
     }
 
-    virtual const char *get_ip_address()
-    {
-        return "127.0.0.1";
-    }
     virtual nsapi_error_t get_ip_address(SocketAddress* address)
     {
         address->set_ip_address("127.0.0.1");

components/wifi/esp8266-driver/ESP8266Interface.cpp

@@ -507,22 +507,6 @@ int ESP8266Interface::disconnect()
     }
 }
 
-const char *ESP8266Interface::get_ip_address()
-{
-    if (_software_conn_stat == IFACE_STATUS_DISCONNECTED) {
-        _esp.uart_enable_input(true);
-    }
-
-    const char *ip_buff = _esp.ip_addr();
-    if (!ip_buff || strcmp(ip_buff, "0.0.0.0") == 0) {
-        ip_buff = NULL;
-    }
-    if (_software_conn_stat == IFACE_STATUS_DISCONNECTED) {
-        _esp.uart_enable_input(false);
-    }
-    return ip_buff;
-}
-
 nsapi_error_t ESP8266Interface::get_ip_address(SocketAddress *address)
 {
     if (_software_conn_stat == IFACE_STATUS_DISCONNECTED) {

components/wifi/esp8266-driver/ESP8266Interface.h

@@ -141,9 +141,6 @@ class ESP8266Interface : public NetworkStack, public WiFiInterface {
      */
     virtual nsapi_error_t get_ip_address(SocketAddress *address);
 
-    MBED_DEPRECATED_SINCE("mbed-os-5.15", "String-based APIs are deprecated")
-    virtual const char *get_ip_address();
-
     /** Get the internally stored MAC address
      *  @return             MAC address of the interface
      */

features/cellular/framework/API/CellularContext.h

@@ -122,8 +122,6 @@ class CellularContext : public CellularInterface {
     virtual nsapi_error_t set_blocking(bool blocking) = 0;
     virtual NetworkStack *get_stack() = 0;
     virtual nsapi_error_t get_ip_address(SocketAddress *address) = 0;
-    MBED_DEPRECATED_SINCE("mbed-os-5.15", "String-based APIs are deprecated")
-    virtual const char *get_ip_address() = 0;
 
     /** Register callback for status reporting.
      *

features/cellular/framework/AT/AT_CellularContext.cpp

@@ -233,21 +233,6 @@ nsapi_error_t AT_CellularContext::get_ip_address(SocketAddress *address)
 #endif
 }
 
-const char *AT_CellularContext::get_ip_address()
-{
-#if NSAPI_PPP_AVAILABLE
-    return nsapi_ppp_get_ip_addr(_at.get_file_handle());
-#else
-    if (!_stack) {
-        _stack = get_stack();
-    }
-    if (_stack) {
-        return _stack->get_ip_address();
-    }
-    return NULL;
-#endif
-}
-
 char *AT_CellularContext::get_interface_name(char *interface_name)
 {
     if (_cid < 0) {

features/cellular/framework/AT/AT_CellularContext.h

@@ -35,7 +35,6 @@ class AT_CellularContext : public CellularContext {
     virtual nsapi_error_t set_blocking(bool blocking);
     virtual NetworkStack *get_stack();
     virtual nsapi_error_t get_ip_address(SocketAddress *address);
-    virtual const char *get_ip_address();
     virtual char *get_interface_name(char *interface_name);
     virtual void attach(mbed::Callback<void(nsapi_event_t, intptr_t)> status_cb);
     virtual nsapi_error_t connect();

features/cellular/framework/targets/UBLOX/AT/UBLOX_AT_CellularContext.cpp

@@ -299,11 +299,6 @@ void UBLOX_AT_CellularContext::get_next_credentials(char **config)
     }
 }
 
-const char *UBLOX_AT_CellularContext::get_gateway()
-{
-    return get_ip_address();
-}
-
 nsapi_error_t UBLOX_AT_CellularContext::get_gateway(SocketAddress *addr)
 {
     return get_ip_address(addr);