Merge pull request #1420 from dchohfi/dchohfi/prevent-headers-lost

Prevent request headers from getting lost

Author: designatednerd

Sources/Apollo/JSONRequest.swift

@@ -98,7 +98,7 @@ open class JSONRequest<Operation: GraphQLOperation>: HTTPRequest<Operation> {
     case .GET:
       let transformer = GraphQLGETTransformer(body: body, url: self.graphQLEndpoint)
       if let urlForGet = transformer.createGetURL() {
-        request = URLRequest(url: urlForGet)
+        request.url = urlForGet
         request.httpMethod = GraphQLHTTPMethod.GET.rawValue
       } else {
         throw GraphQLHTTPRequestError.serializedQueryParamsMessageError

Tests/ApolloTests/AutomaticPersistedQueriesTests.swift

@@ -388,6 +388,7 @@ class AutomaticPersistedQueriesTests: XCTestCase {
     let provider = LegacyInterceptorProvider(client: mockClient)
     let network = RequestChainNetworkTransport(interceptorProvider: provider,
                                                endpointURL: self.endpoint,
+                                               additionalHeaders: ["Authentication": "Bearer 1234"],
                                                useGETForQueries: true)
 
     let expectation = self.expectation(description: "Query sent")
@@ -403,6 +404,7 @@ class AutomaticPersistedQueriesTests: XCTestCase {
 
     XCTAssertEqual(request.url?.host, network.endpointURL.host)
     XCTAssertEqual(request.httpMethod, "GET")
+    XCTAssertEqual(request.allHTTPHeaderFields!["Authentication"], "Bearer 1234")
 
     try self.validateUrlParams(with: request,
                                query: query,