Diff: HTTP_SERVER.cpp

Revision:

14:a16cdcd098d7

Parent:

13:483b2b1a6471

Child:

15:9b2cfbaf1c12

--- a/HTTP_SERVER.cpp	Sun Mar 04 19:08:36 2018 +0000
+++ b/HTTP_SERVER.cpp	Fri Mar 16 22:45:47 2018 +0000
@@ -1,15 +1,52 @@
 #include "HTTP_SERVER.h"
 #include "string"
 #ifndef HTTP_SERVER_DEBUG
-//#define DEBUG
+#define HTTP_SERVER_DEBUG
 #endif
+/*
+LOG:
+"sys_thread_new number error" is occurred when an instance of this class is instantiated in main() in main.cpp
+This case wasn't occured with the APIs of mbed os 2.
+So, inserting some DEBUG_PRINT_LINE, and finding where is wrong. 
+
+The error seems to be occurred in init() in the above case.
+
+TCP_PORT=80 redefine
+tcp_port->port
+
+The error seems to be occurred while ethernet->connect() is proceesing.
+
+The error seems to be caused by something wrong with stack.
+When modify (char* -> char) req_buf[1024] in the .h file,
+the error message became: "CMSIS-RTOS error: Stack underflow (status: 0x1, task ID: 0x10002678, task name: )"
+UNDERFLOW means that there is a value which is too fine(or high definition) to express.
+(Why was the type char*?)
+
+char req_buf[1024] was not uesed anywhere...
+so it was deleted.
+
+bool deep-alive was also not uesed so deleted it.
+
+After doing those above, the error massage was DERAYED but 
+it even now occur.
+and when some array stack(such like char buffer[1024] -> [256])'s size was decreased,
+the message was more derayed.
+Therefore it can be thought that the error is causeed by stack overflow.
+*/
 
 namespace HTTP_SERVER
 {
+void DEBUG_PRINT_NAME()
+{
+#ifdef HTTP_SERVER_DEBUG
+    printf("(DEBUG LINE: HTTP_SERVER) ");
+#endif
+}
+
 void DEBUG_PRINT_LINE(const char* arg_line)
 {
 #ifdef HTTP_SERVER_DEBUG
-    printf("(HTTP_SERVER) ")
+    DEBUG_PRINT_NAME();
     printf(arg_line);
     printf("\r\n");
 #endif
@@ -18,7 +55,7 @@
 void DEBUG_PRINT_LINE(const char* arg_line, T arg_t)
 {
 #ifdef HTTP_SERVER_DEBUG
-    printf("(HTTP_SERVER) ");
+    DEBUG_PRINT_NAME();
     printf(arg_line, arg_t);
     printf("\r\n");
 #endif
@@ -27,7 +64,7 @@
 void DEBUG_PRINT_LINE(const char* arg_line, T1 arg_t1, T2 arg_t2)
 {
 #ifdef HTTP_SERVER_DEBUG
-    printf("(HTTP_SERVER) ");
+    DEBUG_PRINT_NAME();
     printf(arg_line, arg_t1, arg_t2);
     printf("\r\n");
 #endif
@@ -37,11 +74,13 @@
 
 HttpServer::HttpServer()
 {
-    keep_alive = (false);
+    DEBUG_PRINT_LINE("=DEBUG MODE=");
+    net = new EthernetInterface();
+    port = TCP_PORT;
+    backlog  = 1;
+    //keep_alive = (false);
     listening_flag = (false);
-    req_buf[0] = '\0';
-    tcp_port = 80;
-    backlog  = 1;
+    DEBUG_PRINT_LINE("A HTTP_SERVER has been inistantiated.");
 }
 
 HttpServer::~HttpServer()
@@ -49,19 +88,26 @@
 }
 
 bool HttpServer::init()
-{
-
-//  Ethernet Initialization
+{    
+    DEBUG_PRINT_LINE("The HTTP SERVER is being initiated");
+    //  Ethernet Initialization
     //  Ethernet Connecting setup
-    if(net.connect()) {
+    if(net->connect()) {
         printf("(HTTP_SERVER) Error!@EthernetInterface::connect()\r\n");
         return false;
     } else {
-        printf("(HTTP_SERVER) IP Address is %s\r\n", net.getIPAddress());
+        printf("(HTTP_SERVER) IP Address is %s\r\n", net->get_ip_address());
     }
+    
     //  TCP Socket setup
     //  To open Server-side PORT
-    if(server.bind(tcp_port)< 0) {
+    if(server.open(net) < 0) {
+        printf("(HTTP_SERVER) Error!@TCPSocketServer::open()\r\n");
+        return false;
+    } else {
+        printf("(HTTP_SERVER) TCP Server has successfully opened\r\n");
+    }
+    if(server.bind(port)< 0) {
         printf("(HTTP_SERVER) Error!@TCPSocketServer::bind()\r\n");
         return false;
     } else {
@@ -75,32 +121,37 @@
         listening_flag = true;
         printf("(HTTP_SERVER) tcp server is listening...\r\n");
     }
-
+    
     return true;
 }
 
 bool HttpServer::run()
 {
+    
     DigitalOut led1(LED1);
     DigitalOut led2(LED1);
+    bool socket_connection = false;
 
+
+    DEBUG_PRINT_LINE("The HTTP SERVER is starting running");
     while (listening_flag) {
         led1 = true;
         //  blocking mode (never timeout)
         //  waiting client connection
         printf("(HTTP_SERVER) waiting connection\r\n");
-        if(server.accept(tcpcon) < 0) {
+        if(server.accept(&client_socket, &client_address) < 0) {
             printf("(HTTP_SERVER) failed to accept connection.\r\n");
             return -1;
         } else {
-            printf("(HTTP_SERVER) connection success!\r\nIP: %s\r\n",client.get_address());
+            printf("(HTTP_SERVER) connection success!\r\nIP: %s\r\n",client_address.get_ip_address());
+            socket_connection = true;
             led2 = true;
         }
         //  When conected
-        while(client.is_connected()) {
+        while (socket_connection) {
             printf("(HTTP_SERVER) connected\r\n");
 
-            char buffer[1024]   = {0};
+            char buffer[256]   = {0};
             char* httpmethod    = NULL;
             char* filepath      = NULL;
             char* http_ver      = NULL;
@@ -110,8 +161,8 @@
             //
             //  Request Analysis
             //
-            DEBUG_PRINT_LINE("DEBUG MODE");
-            switch(client.receive(buffer, 1023)) {
+            DEBUG_PRINT_LINE("DEBUG MODE@Request Analysis");
+            switch(client_socket.recv(buffer, 255)) {
                 case 0:
                     DEBUG_PRINT_LINE("recieved buffer is empty.");
                     msger.setStatusLine(400, "No Request");
@@ -119,6 +170,7 @@
                     httpmethod    = NULL;
                     filepath      = NULL;
                     http_ver      = NULL;
+                    socket_connection = false;
                     break;
                 case -1:
                     DEBUG_PRINT_LINE("failed to read data from client.");
@@ -127,6 +179,7 @@
                     httpmethod    = NULL;
                     filepath      = NULL;
                     http_ver      = NULL;
+                    socket_connection = false;
                     break;
                 default:
                     DEBUG_PRINT_LINE("Recieved Data: %d",strlen(buffer));
@@ -150,30 +203,30 @@
 
                 //  file calibration
                 DEBUG_PRINT_LINE("file opening");
-                fhandl.open(filepath,"rb");
-                if(fhandl.arrival()) {
+                fhndl.open(filepath,"rb");
+                if(fhndl.arrival()) {
                     msger.setStatusLine(200, "OK");
-                    if(msger.setHeaderField("Content-Length", fhandl.getFileSize()))    DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
+                    if(msger.setHeaderField("Content-Length", fhndl.getFileSize()))     DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
                     if(msger.setHeaderField("Connection", "keep-alive"))                DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
                 } else {
                     if(msger.setStatusLine(404, "NOT FOUND"))                           DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
                     if(msger.setHeaderField("Connection", "Close"))                     DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
                     DEBUG_PRINT_LINE("NOT FOUND");
                 }
-                if(         !strcmp(fhandl.getSuffix(), "htm" ) ||
-                            !strcmp(fhandl.getSuffix(), "HTM" ) ||
-                            !strcmp(fhandl.getSuffix(), "html") ||
-                            !strcmp(fhandl.getSuffix(), "HTML")){
+                if(         !strcmp(fhndl.getSuffix(), "htm" ) ||
+                            !strcmp(fhndl.getSuffix(), "HTM" ) ||
+                            !strcmp(fhndl.getSuffix(), "html") ||
+                            !strcmp(fhndl.getSuffix(), "HTML")) {
                     if(msger.setHeaderField("Content-Type", "text/html"))               DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
-                } else if(  !strcmp(fhandl.getSuffix(), "js"  )){
+                } else if(  !strcmp(fhndl.getSuffix(), "js"  )) {
                     if(msger.setHeaderField("Content-Type", "text/javascript"))         DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
-                } else if ( !strcmp(fhandl.getSuffix(), "ico" )){
+                } else if ( !strcmp(fhndl.getSuffix(), "ico" )) {
                     if(msger.setHeaderField("Content-Type", "image/png"))               DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
-                } else if ( !strcmp(fhandl.getSuffix(), "png" ) ||
-                            !strcmp(fhandl.getSuffix(), "PNG" )){
+                } else if ( !strcmp(fhndl.getSuffix(), "png" ) ||
+                            !strcmp(fhndl.getSuffix(), "PNG" )) {
                     if(msger.setHeaderField("Content-Type", "image/png"))               DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
-                } else if ( !strcmp(fhandl.getSuffix(), "jpg" ) ||
-                            !strcmp(fhandl.getSuffix(), "JPG" )){
+                } else if ( !strcmp(fhndl.getSuffix(), "jpg" ) ||
+                            !strcmp(fhndl.getSuffix(), "JPG" )) {
                     if(msger.setHeaderField("Content-Type", "image/jpg"))               DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
                 } else {
                     msger.setStatusLine(406, "not acceptable");
@@ -183,21 +236,22 @@
                 if(msger.setHeaderField("Keep-Alive", "timeouit=15"))                   DEBUG_PRINT_LINE("buffer over flow @ ResponseMessenger");
 
                 //  send response
-                msger.sendHTTPResponse(tcpcon, fhandl);
+                msger.sendHTTPResponse(client_socket, fhndl);
 
                 //file close
-                if( fhandl.close()== 0)
+                if( fhndl.close()== 0)
                     DEBUG_PRINT_LINE("file has closed");
                 else if(EOF)
                     DEBUG_PRINT_LINE("failed to close the file");
-                    
+
                 msger.resetHeader();
                 DEBUG_PRINT_LINE("echo back done.");
             }
             if (httpmethod == NULL) {
-                msger.sendHTTPResponse(tcpcon);
+                msger.sendHTTPResponse(client_socket);
                 msger.resetHeader();
                 DEBUG_PRINT_LINE("echo back done.");
+                socket_connection = false;
             }
             printf("(HTTP_SERVER) Response to Request has done\r\n");
             //
@@ -205,11 +259,12 @@
             //
         }
         printf("(HTTP_SERVER) close connection.\r\ntcp server is listening...\r\n");
-        client.close();
+        client_socket.close();
         led2 = false;
     }
     server.close();
     listening_flag = false;
     led1 = false;
     return 0;
-}
\ No newline at end of file
+    
+}