fix(frontend-canister): allow overwriting default HTTP headers

CHANGELOG.md

@@ -4,6 +4,22 @@
 
 ## DFX
 
+### fix(frontend-canister): Allow overwirting default HTTP Headers for assets in frontend canister 
+
+Allows to overwrite `Content-Type`, `Content-Encoding`, and `Cache-Control` HTTP headers with custom values via `.ic-assets.json5` config file. Notice this requires using correct capitalization, example `.ic-assets.json5` file:
+```json5
+[
+    {
+        "match": "web-gz.data.gz",
+        "headers": {
+            "Content-Type": "application/octet-stream",
+            "Content-Encoding": "gzip"
+        }
+    },
+]
+```
+
+
 ### fix: Save SNS canister IDs
 
 SNS canister IDs were not being parsed reliably.  Now the candid file is being specified explicitly, which resolves the issue in at least some cases.

src/canisters/frontend/ic-certified-assets/src/state_machine.rs

@@ -709,19 +709,19 @@ fn build_ok(
     callback: Func,
     etags: Vec<Hash>,
 ) -> HttpResponse {
-    let mut headers = vec![("Content-Type".to_string(), asset.content_type.to_string())];
+    let mut headers = HashMap::from([("Content-Type".to_string(), asset.content_type.to_string())]);
     if enc_name != "identity" {
-        headers.push(("Content-Encoding".to_string(), enc_name.to_string()));
+        headers.insert("Content-Encoding".to_string(), enc_name.to_string());
     }
     if let Some(head) = certificate_header {
-        headers.push(head);
+        headers.insert(head.0, head.1);
     }
     if let Some(max_age) = asset.max_age {
-        headers.push(("Cache-Control".to_string(), format!("max-age={}", max_age)));
+        headers.insert("Cache-Control".to_string(), format!("max-age={}", max_age));
     }
     if let Some(arg_headers) = asset.headers.as_ref() {
         for (k, v) in arg_headers {
-            headers.push((k.to_owned(), v.to_owned()));
+            headers.insert(k.to_owned(), v.to_owned());
         }
     }
 
@@ -731,16 +731,16 @@ fn build_ok(
     let (status_code, body) = if etags.contains(&enc.sha256) {
         (304, RcBytes::default())
     } else {
-        headers.push((
+        headers.insert(
             "ETag".to_string(),
             format!("\"{}\"", hex::encode(enc.sha256)),
-        ));
+        );
         (200, enc.content_chunks[chunk_index].clone())
     };
 
     HttpResponse {
         status_code,
-        headers,
+        headers: headers.into_iter().collect::<_>(),
         body,
         streaming_strategy,
     }