Removed the content length as this is a possible cause of the 'The pr…

…ovided 'x-amz-content-sha256' header does not match what was computed.' error (checkout aws/aws-sdk-go#148 (comment) )
terwanerik · Apr 18, 2019 · f0b5cb4 · f0b5cb4
1 parent 5752320
commit f0b5cb4
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/Sources/S3Signer/S3Signer+Private.swift b/Sources/S3Signer/S3Signer+Private.swift
@@ -148,7 +148,6 @@ extension S3Signer {
         }
 
         if httpMethod == .PUT || httpMethod == .DELETE {
-            updatedHeaders["content-length"] = payload.size()
             if httpMethod == .PUT && url.pathExtension != "" {
                 updatedHeaders["content-type"] = (MediaType.fileExtension(url.pathExtension) ?? .plainText).description
             }

diff --git a/Tests/S3Tests/S3SignerAWSTests.swift b/Tests/S3Tests/S3SignerAWSTests.swift
@@ -96,9 +96,7 @@ class S3SignerAWSTests: BaseTestCase {
                                           payload: .bytes(randomBytesMessage),
                                           dates: overridenDate)
 
-        XCTAssertNotNil(headers["Content-Length"].first)
         XCTAssertNotNil(headers["Content-Type"].first)
-        XCTAssertEqual(headers["Content-Length"].first, Payload.bytes(randomBytesMessage).size())
         XCTAssertEqual(headers["Content-Type"].first, "text/plain")
     }