aboutsummaryrefslogtreecommitdiff
path: root/src/api/s3_put.rs
diff options
context:
space:
mode:
authorQuentin <quentin@deuxfleurs.fr>2020-12-10 20:12:56 +0100
committerQuentin <quentin@deuxfleurs.fr>2020-12-10 20:12:56 +0100
commite8c12072cefa37d9aec023fd6087b2d190ee3e4c (patch)
tree88b15bfe703d2454d99550010ad04934513d25a4 /src/api/s3_put.rs
parent51d0c14e440f00f24dbed6c3bce915a183a2bb65 (diff)
parent022b386a5085cad79d649a82846c41cad730920b (diff)
downloadgarage-e8c12072cefa37d9aec023fd6087b2d190ee3e4c.tar.gz
garage-e8c12072cefa37d9aec023fd6087b2d190ee3e4c.zip
Merge branch 'master' into feature/website
Diffstat (limited to 'src/api/s3_put.rs')
-rw-r--r--src/api/s3_put.rs17
1 files changed, 16 insertions, 1 deletions
diff --git a/src/api/s3_put.rs b/src/api/s3_put.rs
index 9c4d625c..c42309b2 100644
--- a/src/api/s3_put.rs
+++ b/src/api/s3_put.rs
@@ -428,6 +428,21 @@ pub async fn handle_complete_multipart_upload(
_ => unreachable!(),
};
+ // ETag calculation: we produce ETags that have the same form as
+ // those of S3 multipart uploads, but we don't use their actual
+ // calculation for the first part (we use random bytes). This
+ // shouldn't impact compatibility as the S3 docs specify that
+ // the ETag is an opaque value in case of a multipart upload.
+ // See also: https://teppen.io/2018/06/23/aws_s3_etags/
+ let num_parts = version.blocks().last().unwrap().part_number
+ - version.blocks().first().unwrap().part_number
+ + 1;
+ let etag = format!(
+ "{}-{}",
+ hex::encode(&rand::random::<[u8; 16]>()[..]),
+ num_parts
+ );
+
// TODO: check that all the parts that they pretend they gave us are indeed there
// TODO: when we read the XML from _req, remember to check the sha256 sum of the payload
// against the signed x-amz-content-sha256
@@ -442,7 +457,7 @@ pub async fn handle_complete_multipart_upload(
ObjectVersionMeta {
headers,
size: total_size,
- etag: "".to_string(), // TODO
+ etag: etag,
},
version.blocks()[0].hash,
));