commit 5f6395256650bd1a58e0617a2d53e85d6c5bfe6c
author Morteza Namvar <morteza.namvar@savoirfairelinux.com> Tue Jul 04 17:08:58 2023 -0400
committer Morteza Namvar <morteza.namvar@savoirfairelinux.com> Tue Jul 04 17:12:01 2023 -0400
tree 247b4d201357f4606dcaae67ef105d9d96dad82c
parent 2a898d47bf72b9ed0d2d37a241fb20154f1c8390

refactor and cleanup

Change-Id: Ia59c83978c26cebe060a301ec37bacd805d36ef5

src/ip_utils.cpp

diff --git a/src/ip_utils.cpp b/src/ip_utils.cpp
index 494dc9b..0b2ded5 100644
--- a/src/ip_utils.cpp
+++ b/src/ip_utils.cpp
@@ -203,7 +203,7 @@
 {
     char localHostBuf[INET_ADDRSTRLEN];
     if (ip_utils::getHostName(localHostBuf, INET_ADDRSTRLEN) < 0) {
-        JAMI_WARN("Couldn't find local host");
+        // JAMI_WARN("Couldn't find local host");
         return {};
     } else {
         return IpAddr(ip_utils::getGateway(localHostBuf, ip_utils::subnet_mask::prefix_24bit));
@@ -227,10 +227,10 @@
     const pj_str_t pjname(sip_utils::CONST_PJ_STR(name));
     auto status = pj_getaddrinfo(family, &pjname, &addr_num, res);
     if (status != PJ_SUCCESS) {
-        JAMI_ERR("Error resolving %.*s : %s",
-                 (int) name.size(),
-                 name.data(),
-                 sip_utils::sip_strerror(status).c_str());
+        // JAMI_ERR("Error resolving %.*s : %s",
+        //          (int) name.size(),
+        //          name.data(),
+        //          sip_utils::sip_strerror(status).c_str());
         return ipList;
     }
 
@@ -268,14 +268,14 @@
     if (status == PJ_SUCCESS) {
         return ip_addr;
     }
-    JAMI_WARN("Could not get preferred address familly (%s)",
-              (family == pj_AF_INET6()) ? "IPv6" : "IPv4");
+    // JAMI_WARN("Could not get preferred address familly (%s)",
+    //           (family == pj_AF_INET6()) ? "IPv6" : "IPv4");
     family = (family == pj_AF_INET()) ? pj_AF_INET6() : pj_AF_INET();
     status = pj_gethostip(family, ip_addr.pjPtr());
     if (status == PJ_SUCCESS) {
         return ip_addr;
     }
-    JAMI_ERR("Could not get local IP");
+    // JAMI_ERR("Could not get local IP");
     return ip_addr;
 }
 
@@ -292,14 +292,14 @@
 
     int fd = socket(unix_family, SOCK_DGRAM, 0);
     if (fd < 0) {
-        JAMI_ERR("Could not open socket: %m");
+        // JAMI_ERR("Could not open socket: %m");
         return addr;
     }
 
     if (unix_family == AF_INET6) {
         int val = family != pj_AF_UNSPEC();
         if (setsockopt(fd, IPPROTO_IPV6, IPV6_V6ONLY, (void*) &val, sizeof(val)) < 0) {
-            JAMI_ERR("Could not setsockopt: %m");
+            // JAMI_ERR("Could not setsockopt: %m");
             close(fd);
             return addr;
         }
@@ -329,7 +329,7 @@
 
     DWORD dwRetval = getaddrinfo(interface.c_str(), "0", &hints, &result);
     if (dwRetval != 0) {
-        JAMI_ERR("getaddrinfo failed with error: %lu", dwRetval);
+        // JAMI_ERR("getaddrinfo failed with error: %lu", dwRetval);
         return addr;
     }
 
@@ -375,7 +375,7 @@
     }
 
 #else
-    JAMI_ERR("Not implemented yet. (iphlpapi.h problem)");
+    // JAMI_ERR("Not implemented yet. (iphlpapi.h problem)");
 #endif
     return ifaceList;
 }