[HTTPCLIENT-2061] Accept-Encoding lost on redirect

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.0
• Fix Version/s: 5.0.1
• Component/s: HttpClient (classic)
• Assignee: Unassigned
• Reporter: Jesse Wilson
• Created: 15/Mar/20 02:05
• Updated: 17/Mar/20 13:52
• Resolved: 17/Mar/20 13:52

Description

Make an HTTP request that is redirected. The HTTP headers of the redirect request does not include an `Accept-Encoding` header.

import okhttp3.mockwebserver.MockResponse
import okhttp3.mockwebserver.MockWebServer
import org.apache.hc.client5.http.classic.methods.HttpGet
import org.apache.hc.client5.http.impl.classic.HttpClients
import org.assertj.core.api.Assertions.assertThat
import org.junit.After
import org.junit.Rule
import org.junit.Test

class ApacheHttpTest {
  @JvmField @Rule val server = MockWebServer()

  private val httpClient = HttpClients.createDefault()

  @After fun tearDown() {
    httpClient.close()
  }

  @Test fun redirect() {
    server.enqueue(MockResponse()
        .setResponseCode(301)
        .addHeader("Location", "/a"))

    server.enqueue(MockResponse())

    val request = HttpGet(server.url("/").toUri())

    httpClient.execute(request).use { response ->
      assertThat(response.code).isEqualTo(200)
    }

    val recordedRequest1 = server.takeRequest()
    assertThat(recordedRequest1.getHeader("Accept-Encoding")).isEqualTo("gzip, x-gzip, deflate")

    val recordedRequest2 = server.takeRequest()
    assertThat(recordedRequest2.getHeader("Accept-Encoding")).isEqualTo("gzip, x-gzip, deflate")
  }
}