Reset response and request data after completion.

SmingHub · Jun 26, 2017 · fab7b5b · fab7b5b
1 parent d44599d
commit fab7b5b
Show file tree

Hide file tree

Showing 6 changed files with 33 additions and 1 deletion.
diff --git a/Sming/SmingCore/Network/Http/HttpRequest.cpp b/Sming/SmingCore/Network/Http/HttpRequest.cpp
@@ -239,6 +239,16 @@ HttpRequest* HttpRequest::onRequestComplete(RequestCompletedDelegate delegateFun
 	return this;
 }
 
+void HttpRequest::reset()
+{
+	headers.clear();
+	postParams.clear();
+	if(queryParams != NULL) {
+		delete queryParams;
+		queryParams = NULL;
+	}
+}
+
 #ifndef SMING_RELEASE
 String HttpRequest::toString() {
 	String content = "";

diff --git a/Sming/SmingCore/Network/Http/HttpRequest.h b/Sming/SmingCore/Network/Http/HttpRequest.h
@@ -116,6 +116,8 @@ class HttpRequest {
 	HttpRequest* onBody(RequestBodyDelegate delegateFunction);
 	HttpRequest* onRequestComplete(RequestCompletedDelegate delegateFunction);
 
+	void reset();
+
 #ifndef SMING_RELEASE
 	/**
 	 * @brief Tries to present a readable version of the current request values

diff --git a/Sming/SmingCore/Network/Http/HttpResponse.cpp b/Sming/SmingCore/Network/Http/HttpResponse.cpp
@@ -180,3 +180,11 @@ bool HttpResponse::sendDataStream( IDataSourceStream * newDataStream , String re
     return true;
 }
 
+void HttpResponse::reset()
+{
+	headers.clear();
+	if(stream != NULL) {
+		delete stream;
+		stream = NULL;
+	}
+}
diff --git a/Sming/SmingCore/Network/Http/HttpResponse.h b/Sming/SmingCore/Network/Http/HttpResponse.h
@@ -81,6 +81,8 @@ class HttpResponse {
 	// Send Datastream, can be called with Classes derived from
 	bool sendDataStream( IDataSourceStream * newDataStream , String reqContentType = "" );
 
+	void reset();
+
 public:
 	int code;
 	HttpHeaders headers;

diff --git a/Sming/SmingCore/Network/Http/HttpServerConnection.cpp b/Sming/SmingCore/Network/Http/HttpServerConnection.cpp
@@ -173,6 +173,11 @@ int HttpServerConnection::staticOnHeadersComplete(http_parser* parser)
 	int error = 0;
 	connection->request.setHeaders(connection->requestHeaders);
 
+	connection->lastWasValue = true;
+	connection->lastData = "";
+	connection->currentField  = "";
+	connection->requestHeaders.clear();
+
 	if(connection->resource != NULL && connection->resource->onHeadersComplete) {
 		error = connection->resource->onHeadersComplete(*connection, connection->request, connection->response);
 	}
@@ -406,6 +411,11 @@ void HttpServerConnection::onReadyToSendData(TcpConnectionEvent sourceEvent)
 		setTimeOut(1); // decrease the timeout to 1 tick
 	}
 
+	if(state == eHCS_Sent) {
+		response.reset();
+		request.reset();
+	}
+
 	TcpClient::onReadyToSendData(sourceEvent);
 }
 

diff --git a/Sming/SmingCore/Network/Http/HttpServerConnection.h b/Sming/SmingCore/Network/Http/HttpServerConnection.h
@@ -97,7 +97,7 @@ class HttpServerConnection: public TcpClient
 	String lastData = "";
 	String currentField  = "";
 
-	BodyParsers* bodyParsers;
+	BodyParsers* bodyParsers = NULL;
 	HttpBodyParserDelegate bodyParser;
 };