Merge pull request #2 from MMP0/patch-1

Add missing CRLF epilogue
FelixHerrmann · Feb 1, 2022 · 19f4e7a · 19f4e7a
2 parents 8652854 + fa26517
commit 19f4e7a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/Sources/MultipartFormData/MultipartFormData.swift b/Sources/MultipartFormData/MultipartFormData.swift
@@ -120,7 +120,7 @@ extension MultipartFormData {
         let bodyData = body
             .map { ._dash + boundary._asciiData + ._crlf + $0._data + ._crlf }
             .reduce(Data(), +)
-        return bodyData + ._dash + boundary._asciiData + ._dash
+        return bodyData + ._dash + boundary._asciiData + ._dash + ._crlf
     }
 }
 

diff --git a/Tests/MultipartFormDataTests/MultipartFormDataTests.swift b/Tests/MultipartFormDataTests/MultipartFormDataTests.swift
@@ -39,7 +39,7 @@ final class MultipartFormDataTests: XCTestCase {
             "Content-Type: application/octet-stream",
             "",
             "",
-            "--test--"
+            "--test--\r\n"
         ].joined(separator: "\r\n").utf8)
         XCTAssertEqual(multipartFormData.httpBody, expectedBody)
     }
@@ -69,7 +69,7 @@ final class MultipartFormDataTests: XCTestCase {
             "Content-Type: application/octet-stream",
             "",
             "",
-            "--test--"
+            "--test--\r\n"
         ].joined(separator: "\r\n")
         XCTAssertEqual(multipartFormData.debugDescription, expectedDescription)
     }

diff --git a/Tests/MultipartFormDataTests/URLRequestTests.swift b/Tests/MultipartFormDataTests/URLRequestTests.swift
@@ -28,7 +28,7 @@ final class URLRequestTests: XCTestCase {
             "Content-Disposition: form-data; name=\"a\"",
             "",
             "",
-            "--test--",
+            "--test--\r\n",
         ].joined(separator: "\r\n").utf8)
         XCTAssertEqual(request.httpBody, expectedBody)
     }