fix(HTTP): error boundary in HTTP-Header, Content-Type

see `https://www.rfc-editor.org/rfc/rfc2046#section-5.1`

Poco carries extra ''", which can cause nginx to return 'Malformed multipart message', upload fail.

Author: HR1025

Net/src/HTMLForm.cpp

@@ -219,9 +219,8 @@ void HTMLForm::prepareSubmit(HTTPRequest& request, int options)
 		{
 			_boundary = MultipartWriter::createBoundary();
 			std::string ct(_encoding);
-			ct.append("; boundary=\"");
+			ct.append("; boundary=");
 			ct.append(_boundary);
-			ct.append("\"");
 			request.setContentType(ct);
 		}
 		if (request.getVersion() == HTTPMessage::HTTP_1_0)

Net/testsuite/src/HTMLFormTest.cpp

@@ -254,7 +254,7 @@ void HTMLFormTest::testReadMultipart()
 		"--MIME_boundary_0123456789--\r\n"
 	);
 	HTTPRequest req("POST", "/form.cgi");
-	req.setContentType(HTMLForm::ENCODING_MULTIPART + "; boundary=\"MIME_boundary_0123456789\"");
+	req.setContentType(HTMLForm::ENCODING_MULTIPART + "; boundary=MIME_boundary_0123456789");
 	StringPartHandler sah;
 	HTMLForm form(req, istr, sah);
 	assertTrue (form.size() == 4);
@@ -309,9 +309,8 @@ void HTMLFormTest::testSubmit3()
 	HTTPRequest req("POST", "/form.cgi", HTTPMessage::HTTP_1_1);
 	form.prepareSubmit(req);
 	std::string expCT(HTMLForm::ENCODING_MULTIPART);
-	expCT.append("; boundary=\"");
+	expCT.append("; boundary=");
 	expCT.append(form.boundary());
-	expCT.append("\"");
 	assertTrue (req.getContentType() == expCT);
 	assertTrue (req.getChunkedTransferEncoding());
 }
@@ -343,9 +342,8 @@ void HTMLFormTest::testSubmit5()
 	HTTPRequest req("POST", "/form.cgi", HTTPMessage::HTTP_1_1);
 	form.prepareSubmit(req, HTMLForm::OPT_USE_CONTENT_LENGTH);
 	std::string expCT(HTMLForm::ENCODING_MULTIPART);
-	expCT.append("; boundary=\"");
+	expCT.append("; boundary=");
 	expCT.append(form.boundary());
-	expCT.append("\"");
 	assertTrue (req.getContentType() == expCT);
 	assertTrue (req.getContentLength() == 403);
 }
@@ -395,7 +393,7 @@ void HTMLFormTest::testFieldLimitMultipart()
 		"--MIME_boundary_0123456789--\r\n"
 	);
 	HTTPRequest req("POST", "/form.cgi");
-	req.setContentType(HTMLForm::ENCODING_MULTIPART + "; boundary=\"MIME_boundary_0123456789\"");
+	req.setContentType(HTMLForm::ENCODING_MULTIPART + "; boundary=MIME_boundary_0123456789");
 	StringPartHandler sah;
 	HTMLForm form;
 	form.setFieldLimit(3);

Net/testsuite/src/MediaTypeTest.cpp

@@ -47,7 +47,7 @@ void MediaTypeTest::testParse()
 	assertTrue (mt3.getParameter("param1") == "value1");
 	assertTrue (mt3.getParameter("PARAM2") == "value 2");
 
-	MediaType mt4("multipart/mixed; boundary=\"MIME_boundary_01234567\"");
+	MediaType mt4("multipart/mixed; boundary=MIME_boundary_01234567");
 	assertTrue (mt4.getType() == "multipart");
 	assertTrue (mt4.getSubType() == "mixed");
 	assertTrue (mt4.parameters().size() == 1);

Net/testsuite/src/MessageHeaderTest.cpp

@@ -309,7 +309,7 @@ void MessageHeaderTest::testSplitParameters()
 	assertTrue (p.size() == 1);
 	assertTrue (p["boundary"] == "MIME_boundary_01234567");
 
-	s = "multipart/related; boundary=\"MIME_boundary_76543210\"";
+	s = "multipart/related; boundary=MIME_boundary_76543210";
 	MessageHeader::splitParameters(s, v, p);
 	assertTrue (v == "multipart/related");
 	assertTrue (p.size() == 1);