adding debbuging statements and minor fix in exit condition of send_response()

Author: StokesAsselborn

src/core/client.cpp

@@ -5,6 +5,8 @@
 #include <fcntl.h>
 #include <unistd.h>
 
+#include <iostream>
+
 void Client::set_handler(Handler* handler)
 {
     handler_ = handler;
@@ -46,6 +48,7 @@ Client::Client(int fd, const sockaddr_in& addr)
 
 {
     set_nonblocking();
+    std::cout << "Client constructor called" << std::endl;
 }
 
 Client::~Client()
@@ -56,6 +59,7 @@ Client::~Client()
     if (handler_) {
         delete (handler_);
     }
+    std::cout << "Client destructor called" << std::endl;
 }
 
 void Client::set_nonblocking()

src/core/server.cpp

@@ -231,13 +231,16 @@ void Server::send_response(Client* conn)
     conn->flush_send_bufffer();
 
     if (conn->send_buffer().empty()) {
+        std::cout << "calling on writable" << std::endl;
         conn->on_writable();
     }
-    if (conn->handler_is_done() && conn->send_buffer().empty()) {
+    // if (conn->handler_is_done() && conn->send_buffer().empty()) {
+    if (conn->handler_is_done()) {
         // assuming HTTP/1.0 for testing
         if (epoll_ctl(epoll_fd_, EPOLL_CTL_DEL, conn->fd(), NULL) == -1) {
             throw UnrecoverableError("epoll_ctl", errno);
         }
+        std::cout << "attempting to remove the connection" << std::endl;
         remove_connection(conn);
     }
 }