Se agrega un poco mas de info de debug y se corrige un detalle.

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

diff --git a/Server/src/httpresponse.cpp b/Server/src/httpresponse.cpp
index 562dbf9881f200291da337745bab5ad400ecd610..0177ae35cf44c9b5aef2f0c0f687569af5b4d85e 100644 (file)
--- a/Server/src/httpresponse.cpp
+++ b/Server/src/httpresponse.cpp
@@ -40,48 +40,55 @@ namespace Server {
 
 HTTPResponse::~HTTPResponse(void) {
 #ifdef DEBUG
 
 HTTPResponse::~HTTPResponse(void) {
 #ifdef DEBUG

-       cerr << __FILE__ << ": destructor." << endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": destructor." << endl;

 #endif // DEBUG
 }
 
 #endif // DEBUG
 }
 

-HTTPResponse::HTTPResponse(const string& version):
-               HTTPMessage(version) {
+HTTPResponse::HTTPResponse(const string& body, const string& version):
+               HTTPMessage(body, version) {

 #ifdef DEBUG
 #ifdef DEBUG

-       cerr << __FILE__ << ": version = " << version << endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": body.length = " << body.length()
+               << " | version = " << version << endl;

 #endif // DEBUG
 }
 
 HTTPResponse::HTTPResponse(const HTTPError& error):
                status_code(error.code) {
 #ifdef DEBUG
 #endif // DEBUG
 }
 
 HTTPResponse::HTTPResponse(const HTTPError& error):
                status_code(error.code) {
 #ifdef DEBUG

-       cerr << __FILE__ << ": HTTPError(status_code = " << error.code
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": HTTPError(status_code = " << error.code

                << ", reason = " << HTTPMessage::reason(error.code) << ", desc = " << error.what()
                << ")" << endl;
 #endif // DEBUG
                << ", reason = " << HTTPMessage::reason(error.code) << ", desc = " << error.what()
                << ")" << endl;
 #endif // DEBUG

-       set_body(string("<plaqui><error desc=\"") + error.what() + "\" /></plaqui>");
+       set_body(string("<response desc=\"") + error.what() + "\" />");

 }
 
 HTTPResponse::HTTPResponse(unsigned status_code, const string& body):
                HTTPMessage(body), status_code(status_code) {
 #ifdef DEBUG
 }
 
 HTTPResponse::HTTPResponse(unsigned status_code, const string& body):
                HTTPMessage(body), status_code(status_code) {
 #ifdef DEBUG

-       cerr << __FILE__ << ": status_code = " << status_code
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": status_code = " << status_code

                << " | body.length = " << body.length() << endl;
 #endif // DEBUG
 }
 
 istream& operator>>(istream& is, HTTPResponse& resp)
                << " | body.length = " << body.length() << endl;
 #endif // DEBUG
 }
 
 istream& operator>>(istream& is, HTTPResponse& resp)

-               throw (HTTPResponse::Error, ios::failure) {
+               throw (HTTPResponse::Error, sockerr, ios::failure) {

 #ifdef DEBUG
 #ifdef DEBUG

-       cerr << __FILE__ << ": operator>>()" << endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": operator>>()" << endl;

 #endif // DEBUG
        char buf[BUFSIZ];
        // Obtengo primera línea (request)
        if (!is.getline(buf, BUFSIZ)) {
 #endif // DEBUG
        char buf[BUFSIZ];
        // Obtengo primera línea (request)
        if (!is.getline(buf, BUFSIZ)) {

-               // No hay mas líneas.
-               throw ios::failure("socket closed");
+               // Fin de archivo.
+               throw ios::failure("eof");

        }
 #ifdef DEBUG
        }
 #ifdef DEBUG

-       cerr << __FILE__ << ":\tRecibiendo linea: " << buf << endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ":\tRecibiendo linea: " << buf << endl;

 #endif // DEBUG
        String line(buf);
        // Si es la primera línea, es el request.
 #endif // DEBUG
        String line(buf);
        // Si es la primera línea, es el request.


@@ -90,7 +97,7 @@ istream& operator>>(istream& is, HTTPResponse& resp)
        }
        // Averiguo la versión.
        string::size_type pos = line.find_first_of(String::SPACE_CHARS, 5);
        }
        // Averiguo la versión.
        string::size_type pos = line.find_first_of(String::SPACE_CHARS, 5);

-       string ver = line.substr(5, pos); 
+       string ver = line.substr(5, pos - 5); 

        if ((ver == "1.1") || (ver == "1.0")) {
                resp.version = ver;
        } else {
        if ((ver == "1.1") || (ver == "1.0")) {
                resp.version = ver;
        } else {


@@ -122,13 +129,13 @@ istream& operator>>(istream& is, HTTPResponse& resp)
        return is;
 }
 
        return is;
 }
 

-ostream& operator<<(ostream& os, const HTTPResponse& resp) {
+ostream& operator<<(ostream& os, const HTTPResponse& resp) throw(sockerr) {

 #ifdef DEBUG
 #ifdef DEBUG

-       cerr << __FILE__ << ": operator<<()" << endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": operator<<()" << endl;

 #endif // DEBUG
        os << "HTTP/" << resp.version << " " << resp.status_code << " "
 #endif // DEBUG
        os << "HTTP/" << resp.version << " " << resp.status_code << " "

-               << HTTPMessage::reason(resp.status_code) << "\n\r";
-       // TODO ver que este bien el \r\l
+               << HTTPMessage::reason(resp.status_code) << "\r\n";

        os << static_cast<const HTTPMessage&>(resp);
        return os;
 }
        os << static_cast<const HTTPMessage&>(resp);
        return os;
 }