- Se sobreescribe el método Connection::finish() para que cierre el socket.

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

diff --git a/Server/src/httpmessage.cpp b/Server/src/httpmessage.cpp
index af2875ecfcb441e2c48202033be7db999c135189..ba2b82596de3a251209b7ef77e30c445c2a191be 100644 (file)
--- a/Server/src/httpmessage.cpp
+++ b/Server/src/httpmessage.cpp
@@ -58,7 +58,7 @@ void HTTPMessage::set_body(const string& _body) {
        body = _body;
        if (body.length()) {
                stringstream ss; // TODO ver forma mas linda de convertir
        body = _body;
        if (body.length()) {
                stringstream ss; // TODO ver forma mas linda de convertir

-               ss << (body.length());
+               ss << (body.length()+1); // FIXME No se por que tengo que sumarle 1.

                headers["Accept-Ranges"] = "bytes";
                headers["Content-Length"] = ss.str();
        }
                headers["Accept-Ranges"] = "bytes";
                headers["Content-Length"] = ss.str();
        }


@@ -74,36 +74,33 @@ istream& operator>>(istream& is, HTTPMessage& m) {
 #endif // DEBUG
        char buf[BUFSIZ];
        bool is_header = true;
 #endif // DEBUG
        char buf[BUFSIZ];
        bool is_header = true;

-       // TODO body
-       // Para hacer que reciba bien el body hay que chequear la cabecera
-       // Content-length, si queda tiempo lo haré...
-       //stringstream body_ss;

        while (is.getline(buf, BUFSIZ)) {
                String sbuf(buf);
                sbuf.trim();
                if (sbuf.length()) {
        while (is.getline(buf, BUFSIZ)) {
                String sbuf(buf);
                sbuf.trim();
                if (sbuf.length()) {

-                       if (is_header) {
-                               stringstream ss;
-                               ss << sbuf;
-                               ss >> m.headers;
-                       }// else { TODO body
-                       //      body_ss << sbuf << endl;
-                       //}
+                       stringstream ss;
+                       ss << sbuf;
+                       ss >> m.headers;
+               // Fin de las cabeceras.

                } else {
                } else {

-                       if (is_header) {
-                               // TODO body
-                               // Ver si tiene un Content-Length para saber si esperamos body.
-                               // Si no esperamos body, no hay que hacer otro is.getline()
-                               // porque se queda esperando forever.
-                               is_header = false;
-                               break;
-                       }// else { TODO body
-                       //      body_ss << sbuf << endl;
-                       //}
+                       // Hay Content-Length, entonces hay body (no respeta RFC AFAIK).
+                       if (m.headers.find("Content-Length") != m.headers.end()) {
+                               // Descarta la línea vacía para separar las cabeceras.
+                               is.getline(buf, BUFSIZ);
+                               stringstream ss(m.headers["Content-Length"]);
+                               streamsize size;
+                               ss >> size;
+                               char* buf2 = new char[size+1];
+                               if (is.readsome(buf2, size)) {
+                                       m.body = buf2;
+                               }
+                               delete buf2[];
+                       }
+                       // Después de una línea vacía, haya obtenido el body o no, sale del
+                       // while.
+                       break;

                }
        }
                }
        }

-       // TODO body
-       //m.body = body_ss.str();

        return is;
 }
 
        return is;
 }
 


@@ -122,6 +119,8 @@ string HTTPMessage::reason(unsigned code) {
                        return "OK";
                case BAD_REQUEST:
                        return "Bad Request";
                        return "OK";
                case BAD_REQUEST:
                        return "Bad Request";

+               case NOT_FOUND:
+                       return "Not Found";

                case LENGTH_REQUIRED:
                        return "Length Required";
                case INTERNAL_SERVER_ERROR:
                case LENGTH_REQUIRED:
                        return "Length Required";
                case INTERNAL_SERVER_ERROR:


@@ -131,7 +130,7 @@ string HTTPMessage::reason(unsigned code) {
                case HTTP_VERSION_NOT_SUPPORTED:
                        return "HTTP Version Not Supported";
                default:
                case HTTP_VERSION_NOT_SUPPORTED:
                        return "HTTP Version Not Supported";
                default:

-                       return "";
+                       return "No Reason";

        }
 }
 
        }
 }