Shutdown idle HttpSockets, serve a complete website, etc

src/App.h

@@ -90,6 +90,8 @@ protected:
             static_dispatch(us_ssl_socket_context_on_data, us_socket_context_on_data)(httpServerContext, [](auto *s, char *data, int length) {
                 Data *appData = (Data *) static_dispatch(us_ssl_socket_context_ext, us_socket_context_ext)(static_dispatch(us_ssl_socket_get_context, us_socket_get_context)(s));
 
+                // warning: should NOT reset timer on any data, ONLY reset data on full HTTP requests!
+                // warning: if we are in shutdown state, resetting the timer is a security issue!
                 static_dispatch(us_ssl_socket_timeout, us_socket_timeout)(s, HTTP_IDLE_TIMEOUT_S);
 
                 // onHttpRequest should probably be hard-coded to HttpRouter
@@ -101,16 +103,30 @@ protected:
             });
 
             static_dispatch(us_ssl_socket_context_on_writable, us_socket_context_on_writable)(httpServerContext, [](auto *s) {
+
+                // what if the client
+
+                // I think it's fair to never mind this one -> if we keep writing data after shutting down then that's an issue for us
                 static_dispatch(us_ssl_socket_timeout, us_socket_timeout)(s, HTTP_IDLE_TIMEOUT_S);
 
                 ((HttpSocket<SSL> *) s)->onWritable();
             });
 
+            static_dispatch(us_ssl_socket_context_on_end, us_socket_context_on_end)(httpServerContext, [](auto *s) {
+                std::cout << "Socket was half-closed!" << std::endl;
+            });
+
             static_dispatch(us_ssl_socket_context_on_timeout, us_socket_context_on_timeout)(httpServerContext, [](auto *s) {
 
-                // basically, when any socket times out we want to close it
+                if (static_dispatch(us_ssl_socket_is_shut_down, us_socket_is_shut_down)(s)) {
+                    std::cout << "Forcefully closing socket since shutdown was not answered in time" << std::endl;
+                    static_dispatch(us_ssl_socket_close, us_socket_close)(s);
+                } else {
+                    std::cout << "Shutting down socket now" << std::endl;
+                    static_dispatch(us_ssl_socket_timeout, us_socket_timeout)(s, HTTP_IDLE_TIMEOUT_S);
+                    static_dispatch(us_ssl_socket_shutdown, us_socket_shutdown)(s);
+                }
 
-                std::cout << "The server would now like to close a socket!" << std::endl;
             });
         }
 

src/HttpSocket.h

@@ -113,20 +113,26 @@ struct HttpSocket {
 
         std::string_view chunk = cb(0);
 
+        // kopiera upp till (SSL eller icke-ssl) max copy distance
+
+        // om mer än detta, fortsätt skicka
+
+
+        // this strategy can be simplified to one, we can even have MAX_COPY_DISTANCE_SSL and MAX_COPY_DISTANCE
         if (length < uWS::Loop::MAX_COPY_DISTANCE) {
             // what if the streamer cannot return any data?
             // then it should return something to pause write, and then start it again
             // basically we need throttling
             writeToCorkBufferAndReset(chunk.data(), chunk.length(), length, false);
         } else {
-            // basically finish off the header section and send it as separate syscall (we do not copy anthing in this strategy)
-            writeToCorkBufferAndReset(nullptr, 0, length, true);
+            // copying some data with the headers is a good idea for SSL but probably not for non-SSL
+            writeToCorkBufferAndReset(chunk.data(), uWS::Loop::MAX_COPY_DISTANCE, length, true);
 
             // just assume this went fine
             Data *httpData = (Data *) static_dispatch(us_ssl_socket_ext, us_socket_ext)((SOCKET_TYPE *) this);
 
             // write that off!
-            static_dispatch(us_ssl_socket_write, us_socket_write)((SOCKET_TYPE *) this, chunk.data(), chunk.length(), 0);
+            static_dispatch(us_ssl_socket_write, us_socket_write)((SOCKET_TYPE *) this, chunk.data() + uWS::Loop::MAX_COPY_DISTANCE, chunk.length() - uWS::Loop::MAX_COPY_DISTANCE, 0);
 
             // if offset is at the end, we are done
             if (httpData->offset < length) {

src/Loop.h

@@ -38,7 +38,7 @@ struct Loop {
 
     }
 
-    Loop() : loop(us_create_loop(wakeupCb, preCb, postCb, sizeof(Data))) {
+    Loop() : loop(us_create_loop(1, wakeupCb, preCb, postCb, sizeof(Data))) {
         new (data = (Data *) us_loop_ext(loop)) Data();
     }