From c8b41d5e350354881caa933ab363a2e90f991524 Mon Sep 17 00:00:00 2001
From: Florent Le Coz <louiz@louiz.org>
Date: Sat, 8 Feb 2014 07:52:22 +0100
Subject: Be verbose about IRC server connection failures, and handle them
 properly

---
 src/network/socket_handler.cpp | 19 +++++++++++--------
 src/network/socket_handler.hpp |  3 ++-
 2 files changed, 13 insertions(+), 9 deletions(-)

(limited to 'src/network')

diff --git a/src/network/socket_handler.cpp b/src/network/socket_handler.cpp
index 67adacd..ea674d1 100644
--- a/src/network/socket_handler.cpp
+++ b/src/network/socket_handler.cpp
@@ -23,7 +23,7 @@ SocketHandler::SocketHandler():
     throw std::runtime_error("Could not create socket");
 }
 
-bool SocketHandler::connect(const std::string& address, const std::string& port)
+std::pair<bool, std::string> SocketHandler::connect(const std::string& address, const std::string& port)
 {
   log_info("Trying to connect to " << address << ":" << port);
   struct addrinfo hints;
@@ -35,15 +35,18 @@ bool SocketHandler::connect(const std::string& address, const std::string& port)
 
   struct addrinfo* addr_res;
   const int res = ::getaddrinfo(address.c_str(), port.c_str(), &hints, &addr_res);
-  // Make sure the alloced structure is always freed at the end of the
-  // function
-  utils::ScopeGuard sg([&addr_res](){ freeaddrinfo(addr_res); });
 
   if (res != 0)
     {
-      perror("getaddrinfo");
-      throw std::runtime_error("getaddrinfo failed");
+      log_warning(std::string("getaddrinfo failed: ") + gai_strerror(res));
+      this->close();
+      return std::make_pair(false, gai_strerror(res));
     }
+
+  // Make sure the alloced structure is always freed at the end of the
+  // function
+  utils::ScopeGuard sg([&addr_res](){ freeaddrinfo(addr_res); });
+
   for (struct addrinfo* rp = addr_res; rp; rp = rp->ai_next)
     {
       if (::connect(this->socket, rp->ai_addr, rp->ai_addrlen) == 0)
@@ -51,14 +54,14 @@ bool SocketHandler::connect(const std::string& address, const std::string& port)
           log_info("Connection success.");
           this->connected = true;
           this->on_connected();
-          return true;
+          return std::make_pair(true, "");
         }
       log_info("Connection failed:");
       perror("connect");
     }
   log_error("All connection attempts failed.");
   this->close();
-  return false;
+  return std::make_pair(false, "");
 }
 
 void SocketHandler::set_poller(Poller* poller)
diff --git a/src/network/socket_handler.hpp b/src/network/socket_handler.hpp
index ec63ea7..c27d44c 100644
--- a/src/network/socket_handler.hpp
+++ b/src/network/socket_handler.hpp
@@ -2,6 +2,7 @@
 # define SOCKET_HANDLER_INCLUDED
 
 #include <string>
+#include <utility>
 
 typedef int socket_t;
 
@@ -21,7 +22,7 @@ public:
   /**
    * Connect to the remote server, and call on_connected() if this succeeds
    */
-  bool connect(const std::string& address, const std::string& port);
+  std::pair<bool, std::string> connect(const std::string& address, const std::string& port);
   /**
    * Set the pointer to the given Poller, to communicate with it.
    */
-- 
cgit v1.2.3