Skip to content

Commit caf527f

Browse files
authored
Merge pull request duckdb#104 from carlopi/revert_etag
Revert "Allow multiple cases for ETag / etag"
2 parents fdb164f + 6d97732 commit caf527f

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

extension/httpfs/s3fs.cpp

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -367,15 +367,10 @@ void S3FileSystem::UploadBuffer(S3FileHandle &file_handle, shared_ptr<S3WriteBuf
367367
static_cast<int>(res->status));
368368
}
369369

370-
if (!res->headers.HasHeader("ETag") && !res->headers.HasHeader("etag")) {
370+
if (!res->headers.HasHeader("ETag")) {
371371
throw IOException("Unexpected response when uploading part to S3");
372372
}
373-
374-
if (res->headers.HasHeader("ETag")) {
375-
etag = res->headers.GetHeaderValue("ETag");
376-
} else if (res->headers.HasHeader("etag")) {
377-
etag = res->headers.GetHeaderValue("etag");
378-
}
373+
etag = res->headers.GetHeaderValue("ETag");
379374
} catch (std::exception &ex) {
380375
ErrorData error(ex);
381376
if (error.Type() != ExceptionType::IO && error.Type() != ExceptionType::HTTP) {

0 commit comments

Comments
 (0)