diff options
| author | Alex <lx@deuxfleurs.fr> | 2025-01-27 19:32:19 +0000 |
|---|---|---|
| committer | Alex <lx@deuxfleurs.fr> | 2025-01-27 19:32:19 +0000 |
| commit | a1d081ee840b1727ba1b3f430638a1296738283e (patch) | |
| tree | 1c4822ec7dec299af99afa22a0614408bc74aabb /src/api/s3/get.rs | |
| parent | beedc9fd11d24ccce837cc229ea6c30c980f9baa (diff) | |
| parent | e8fa89e8348522c368eb17e89ab3209e6a734088 (diff) | |
| download | garage-a1d081ee840b1727ba1b3f430638a1296738283e.tar.gz garage-a1d081ee840b1727ba1b3f430638a1296738283e.zip | |
Merge pull request 's3 api: make x-amz-meta-* headers lowercase (fix #844)' (#938) from fix-844 into main
Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/938
Diffstat (limited to 'src/api/s3/get.rs')
| -rw-r--r-- | src/api/s3/get.rs | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/src/api/s3/get.rs b/src/api/s3/get.rs index f5d3cf11..f61aae11 100644 --- a/src/api/s3/get.rs +++ b/src/api/s3/get.rs @@ -68,14 +68,11 @@ fn object_headers( // See: https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingMetadata.html let mut headers_by_name = BTreeMap::new(); for (name, value) in meta_inner.headers.iter() { - match headers_by_name.get_mut(name) { - None => { - headers_by_name.insert(name, vec![value.as_str()]); - } - Some(headers) => { - headers.push(value.as_str()); - } - } + let name_lower = name.to_ascii_lowercase(); + headers_by_name + .entry(name_lower) + .or_insert(vec![]) + .push(value.as_str()); } for (name, values) in headers_by_name { |