summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorent Le Coz <louiz@louiz.org>2014-06-08 15:13:08 +0200
committerFlorent Le Coz <louiz@louiz.org>2014-06-08 15:13:08 +0200
commit8b30e312bbc0728b527941d5b0a6d0f621025ed0 (patch)
tree95373b123e63ab8f35fe9e486e92c4288756d3df
parent349dc06e91bd1b944bd2a1770dba7b7a3a67ce61 (diff)
downloadbiboumi-8b30e312bbc0728b527941d5b0a6d0f621025ed0.tar.gz
biboumi-8b30e312bbc0728b527941d5b0a6d0f621025ed0.tar.bz2
biboumi-8b30e312bbc0728b527941d5b0a6d0f621025ed0.tar.xz
biboumi-8b30e312bbc0728b527941d5b0a6d0f621025ed0.zip
Add a TimedEvent to cancel the connection to a server after 5 seconds
-rw-r--r--src/network/socket_handler.cpp18
-rw-r--r--src/network/socket_handler.hpp5
2 files changed, 23 insertions, 0 deletions
diff --git a/src/network/socket_handler.cpp b/src/network/socket_handler.cpp
index d989623..a2c22e4 100644
--- a/src/network/socket_handler.cpp
+++ b/src/network/socket_handler.cpp
@@ -1,5 +1,6 @@
#include <network/socket_handler.hpp>
+#include <utils/timed_events.hpp>
#include <utils/scopeguard.hpp>
#include <network/poller.hpp>
@@ -26,6 +27,7 @@
#endif
using namespace std::string_literals;
+using namespace std::chrono_literals;
namespace ph = std::placeholders;
@@ -117,6 +119,8 @@ void SocketHandler::connect(const std::string& address, const std::string& port,
|| errno == EISCONN)
{
log_info("Connection success.");
+ TimedEventsManager::instance().cancel("connection_timeout"s +
+ std::to_string(this->socket));
this->poller->add_socket_handler(this);
this->connected = true;
this->connecting = false;
@@ -139,6 +143,12 @@ void SocketHandler::connect(const std::string& address, const std::string& port,
memcpy(&this->addrinfo, rp, sizeof(struct addrinfo));
this->addrinfo.ai_addr = &this->ai_addr;
this->addrinfo.ai_next = nullptr;
+ // If the connection has not succeeded or failed in 5s, we consider
+ // it to have failed
+ TimedEventsManager::instance().add_event(
+ TimedEvent(std::chrono::steady_clock::now() + 5s,
+ std::bind(&SocketHandler::on_connection_timeout, this),
+ "connection_timeout"s + std::to_string(this->socket)));
return ;
}
log_info("Connection failed:" << strerror(errno));
@@ -149,6 +159,12 @@ void SocketHandler::connect(const std::string& address, const std::string& port,
return ;
}
+void SocketHandler::on_connection_timeout()
+{
+ this->close();
+ this->on_connection_failed("connection timed out");
+}
+
void SocketHandler::connect()
{
this->connect(this->address, this->port, this->use_tls);
@@ -273,6 +289,8 @@ void SocketHandler::close()
this->in_buf.clear();
this->out_buf.clear();
this->port.clear();
+ TimedEventsManager::instance().cancel("connection_timeout"s +
+ std::to_string(this->socket));
}
socket_t SocketHandler::get_socket() const
diff --git a/src/network/socket_handler.hpp b/src/network/socket_handler.hpp
index 02311c4..95e38bd 100644
--- a/src/network/socket_handler.hpp
+++ b/src/network/socket_handler.hpp
@@ -86,6 +86,11 @@ public:
*/
void close();
/**
+ * Called by a TimedEvent, when the connection did not succeed or fail
+ * after a given time.
+ */
+ void on_connection_timeout();
+ /**
* Called when the connection is successful.
*/
virtual void on_connected() = 0;