Close #5 - Shutting down the server should now free all resources

Author: fhessel

Diff for: https/HTTPSServer.cpp

@@ -79,20 +79,25 @@ bool HTTPSServer::isRunning() {
 void HTTPSServer::stop() {
 
 	if (_running) {
+		// Set the flag that the server is stopped
+		_running = false;
+
+		// Clean up the connections
+		for(int i = 0; i < _maxConnections; i++) {
+			if (_connections[i] != NULL) {
+				_connections[i]->closeConnection();
+				delete _connections[i];
+			}
+		}
+
+		// Close the actual server socket
 		close(_socket);
 		_socket = -1;
 
-		// Reset the context
+		// Tear down the SSL context
 		SSL_CTX_free(_sslctx);
 		_sslctx = NULL;
 	}
-
-	// Clean up the connections
-	for(int i = 0; i < _maxConnections; i++) {
-		if (_connections[i] != NULL) {
-
-		}
-	}
 }
 
 /**
@@ -110,6 +115,9 @@ void HTTPSServer::setDefaultHeader(std::string name, std::string value) {
  */
 void HTTPSServer::loop() {
 
+	// Only handle requests if the server is still running
+	if(!_running) return;
+
 	// Step 1: Process existing connections
 	// Process open connections and store the index of a free connection
 	// (we might use that later on)