Content-Disposition support is missing

[petrov-aleksey]

Hi,

it seems that Content-Disposition is missing between put and get object.

Test to reproduce

@Test
fun testPutGetObject_contentDisposition(testInfo: TestInfo) {
  val bucket = givenRandomBucketV2()
  val uploadFile = File(UPLOAD_FILE_NAME)
  val contentDisposition = ContentDisposition.formData()
    .name("file")
    .filename("sampleFile.txt")
    .build()
    .toString()

  s3ClientV2.putObject(
    PutObjectRequest.builder()
      .bucket(bucket)
      .key(UPLOAD_FILE_NAME)
      .contentDisposition(contentDisposition)
      .build(),
    RequestBody.fromFile(uploadFile))

  val getObjectResponseResponse = getObjectV2(bucket, UPLOAD_FILE_NAME)

  assertThat(getObjectResponseResponse.response().contentDisposition())
    .isEqualTo(contentDisposition)
}

outputs

org.opentest4j.AssertionFailedError: 
expected: "form-data; name="file"; filename="sampleFile.txt""
 but was: null

[afranken]

@petrov-aleksey thanks for the report and the test.
Looking at our code and the Get/PutObject API, it seems that Content-Disposition isn't the only header we currently do not support...
Expires, Content-Language and Cache-Control are currently also ignored.

I'll look into supporting these headers.

[afranken]

merged feature #1177