Comment by Chris Bailey (JIRA)

The The call to URLSession.dataTask() creates a task with the following constructor:

URLSessionDataTask(session: self, request: r, taskIdentifier: i)
this in turn calls:

init(session: session, request: request, taskIdentifier: taskIdentifier, body: .none)
ie, we create a task with no body.

Its fairly straightforward to fix that - however it doesn't fix the overall test case due to a second problem - on Linux we fail to set the "Content-Type" header value to "application/x-www-form-urlencoded".

I'll raise a PR for the first part to try to get that in as quickly as possible. The second part can be temporarily worked around by setting the header value manually using:

req.setValue("application/x-www-form-urlencoded", forHTTPHeaderField: "Content-Type" )

Comment by Chris Bailey (JIRA)

I've raised PR 633:
#633

Comment by Mike Ferenduros (JIRA)

Related: It looks like in Foundation, uploadTask(with: request, from: data) sends (data ?? request.httpBody) as the request body (although I can't derive this from the docs). Presumably the same applies to URL vs httpBodyStream ❓

Comment by Chris Bailey (JIRA)

Are you referring to
open func uploadTask(with request: URLRequest, from bodyData: Data?, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Swift.Void) -> URLSessionUploadTask

That looks like the only point that Data is an optional, where yes we could fall back to request.httpBody (which it doesn't today.

Comment by Mike Ferenduros (JIRA)

Yep, thats the one.

Would it be useful for me to flesh out the tests with this stuff, and some more validations against httpbin.org for things like auth, redirection etc?

Comment by Chris Bailey (JIRA)

Absolutely - at the moment the tests we have are somewhat limited, so anything you can provide would be hugely valuable.

Comment by Mike Ferenduros (JIRA)

Cool 🙂

Working on it here:
https://github.com/mike-ferenduros/swift-corelibs-foundation/tree/more-urlsession-tests (NB: untested on Linux)

It's pretty early but I hit a few issues:

• 302 redirects are apparently misinterpreting the 'Location' header and hence failing

• The redirect delegate method is getting called after the response is delivered, which I think contradicts the docs (although it's woolly)

• No basic or digest auth

Comment by Andrey Fidrya (JIRA)

Copying from PR 633:
In some situations httpBody is still not sent. Below is the code which reproduces the issue. If "Content-Type" line is uncommented, second dataTask won't send httpBody:

import Foundation
import Dispatch

var req = URLRequest(url: URL(string: "http://httpbin.org/post")!)
req.httpMethod = "POST"
req.httpBody = "a=b&c=d".data(using: .utf8)
//req.setValue("application/x-www-form-urlencoded", forHTTPHeaderField: "Content-Type")

let sesh = URLSession(configuration: URLSessionConfiguration.default)

let dataTask1 = sesh.dataTask(with: req) { data, _, _ in
            print(String(data: data!, encoding: .utf8)!)
}
dataTask1.resume()

let dataTask2 = sesh.dataTask(with: req) { data, _, _ in
            print(String(data: data!, encoding: .utf8)!)
}
dataTask2.resume()

dispatchMain()

Comment by sai Hema k (JIRA)

klassobanieras (JIRA User) This issue seems to be fixed now.Can you please verify and close this ?

Tested as working on both swift 4.2 and 5.1.4