Se arregla el bug que hacia que se cuelgue el cliente al desconectarse.

[z.facultad/75.42/plaqui.git] / Server / src / controlserver.cpp

diff --git a/Server/src/controlserver.cpp b/Server/src/controlserver.cpp
index 4b888493b2264666c733dc3dfc7bdf8242ec18e5..a9cf729fd091c1662b6733c0a0b3b4cb47d914ee 100644 (file)
--- a/Server/src/controlserver.cpp
+++ b/Server/src/controlserver.cpp
@@ -25,44 +25,137 @@
 // $Id$
 //
 
 // $Id$
 //
 

-#include "controlserver.h"
-#include <cstring>
-#include <sstream>
+#include "plaqui/server/controlserver.h"
+#include "plaqui/server/command.h"
+#include "plaqui/server/httperror.h"
+#include "plaqui/server/httpresponse.h"
+//#include <cstring>
+//#include <sstream>
+#ifdef DEBUG
+#      include "plaqui/server/string.h"
+#      include <iostream>
+#endif // DEBUG
+
+using namespace std;

 
 

-using namespace Plaqui;
+namespace PlaQui {
+
+namespace Server {
+
+ControlServer::~ControlServer(void) {
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": destructor." << endl;
+#endif // DEBUG
+}

 
 ControlServer::ControlServer(const sockbuf::sockdesc& sd):
                Connection(sd) {
 #ifdef DEBUG
 
 ControlServer::ControlServer(const sockbuf::sockdesc& sd):
                Connection(sd) {
 #ifdef DEBUG

-       std::cerr << __FILE__ << ": sd = " << sd.sock << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": sd = " << sd.sock << endl;

 #endif // DEBUG
 }
 
 void ControlServer::real_run(void) {
 #endif // DEBUG
 }
 
 void ControlServer::real_run(void) {

-       // FIXME se tiene que ir a la clase para poder frenarlo desde afuera.
-       bool stop = false;
-       char buf[BUFFER_SIZE];
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": real_run()" << endl;
+#endif // DEBUG
+       //char buf[BUFSIZ];

        while (!stop) {
        while (!stop) {

-               stringstream sstr;
-               while (!stop && socket.getline(buf, BUFFER_SIZE)) {
+               Command command;
+               try {
+                       //Glib::Mutex::Lock lock(socket_mutex);
+                       socket >> command;
+               } catch (const ios::failure& e) {
+                       // TODO poner buenos codigos de error.
+                       error(1000000, "Se desconectó.");
+                       return;
+               } catch (const sockerr& e) {
+                       error(e.serrno(), e.errstr());
+                       return;
+               // Si se cerró el socket.
+               //} catch (const ios::failure& e) {
+               //      stop = true;
+               //      continue;
+               // Si hay un error al parsear el comando, se envia una respuesta con el
+               // error.
+               } catch (const HTTPError& e) {
+#ifdef DEBUG
+                       cerr << __FILE__ << "(" << __LINE__ << ")"
+                               << " : real_run() ERROR: status_code = "
+                               << e.code << " | reason = " << HTTPMessage::reason(e.code)
+                               << " | desc = " << e.what() << endl;
+#endif // DEBUG
+                       //Glib::Mutex::Lock lock(socket_mutex);
+                       socket << HTTPResponse(e) << flush;
+                       continue;
+               }
+               // TODO agregar las verificaciones de abajo a HTTPRequest y padres.
+               // Actualizacion: Estoy usando trim() en casi todos lados, no debería
+               // ser necesario.
+/*
+               // Primera línea no vacía (que debe ser el request).
+               bool is_first = true;
+               while (!stop && socket.getline(buf, BUFSIZ)) {

 #ifdef DEBUG
 #ifdef DEBUG

-                       std::cerr << "Reciviendo: " << buf << std::endl;
+                       cerr << __FILE__ << "(" << __LINE__ << ")"
+                       << "  Recibiendo inea: " << buf << endl;

 #endif // DEBUG
 #endif // DEBUG

-                       // Si tiene contenido, lo almaceno en el buffer de comandos.
-                       if (strlen(buf)) {
-                               sstr << buf << endl;
-                       // Si viene la línea vacía, terminan las cabeceras HTTP.
+                       int len = strlen(buf);
+                       // Si tiene un retorno de carro, lo elimina.
+                       if (len && (buf[len-1] == '\r')) {
+                               buf[--len] = '\0';
+                       }
+                       // Si tiene contenido, lo agrego a la información del request.
+                       if (len) {
+                               // Si es la primera línea, es el request.
+                               if (is_first) {
+                                       request.set_request(buf, socket->peerhost(),
+                                                       socket->peerport());
+                                       is_first = false;
+                               } else {
+                                       // TODO request.parse_header(buf);
+                               }
+                       // Si viene la línea vacía

                        } else {
                        } else {

-                               stop = true;
+                               // Si no es la primera, terminan las cabeceras HTTP.
+                               if (!is_first) {
+                                       // Podría ir un break.
+                                       stop = true;
+                                       continue;
+                               }
+                               // Si es la primera, no pasa nada, sigue esperando un request.

                        }
                }
                        }
                }

-               // Manda mensaje a la planta.
-               //dispatch_command(parse_command(sstr.str()));
+*/

 #ifdef DEBUG
 #ifdef DEBUG

-                       std::cerr << "Recivido:" << std::endl << sstr.str() << std::endl;
+               cerr << __FILE__ << "(" << __LINE__ << ")"
+                       << " : real_run() Despachando comando: target = "
+                       << command.get_target() << " | command = " << command.get_command()
+                       << " | args = [" << String::join(command.get_args(), ", ") << "]"
+                       << endl;

 #endif // DEBUG
 #endif // DEBUG

-               // FIXME - hacer respuesta XML.
-               socket << "Ok!" << std::endl;
+               // Manda el comando.
+               command_received(command);

        }
 }
 
        }
 }
 

+void ControlServer::send(const HTTPResponse& response) {
+       //Glib::Mutex::Lock lock(socket_mutex);
+       socket << response << flush;
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": send() Enviado!" << endl;
+#endif // DEBUG
+}
+
+ControlServer::SignalCommandReceived& ControlServer::signal_command_received(void) {
+       return command_received;
+}
+
+} // namespace Server
+
+} // namespace PlaQui
+