From c58e00fe2271d77ede9ccab5a6b317689859ea98 Mon Sep 17 00:00:00 2001
From: Paul Liu <paul@dfinity.org>
Date: Wed, 11 Dec 2024 21:25:41 +0800
Subject: [PATCH] fix(ckbtc): Drop Host header from https outcall request
 (#3115)

Remove the redundant "Host" header to avoid errors in making https
outcall via http/2.

This fixes problem accessing API providers blockstream.info and
mempool.space.
---
 rs/bitcoin/checker/src/providers.rs | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/rs/bitcoin/checker/src/providers.rs b/rs/bitcoin/checker/src/providers.rs
index a7e141c7eaf..c369f2149db 100644
--- a/rs/bitcoin/checker/src/providers.rs
+++ b/rs/bitcoin/checker/src/providers.rs
@@ -119,16 +119,10 @@ impl Provider {
 fn btcscan_request(txid: Txid, max_response_bytes: u32) -> CanisterHttpRequestArgument {
     let host = "btcscan.org";
     let url = format!("https://{}/api/tx/{}/raw", host, txid);
-    let request_headers = vec![
-        HttpHeader {
-            name: "Host".to_string(),
-            value: format!("{host}:443"),
-        },
-        HttpHeader {
-            name: "User-Agent".to_string(),
-            value: "bitcoin_inputs_collector".to_string(),
-        },
-    ];
+    let request_headers = vec![HttpHeader {
+        name: "User-Agent".to_string(),
+        value: "bitcoin_inputs_collector".to_string(),
+    }];
     CanisterHttpRequestArgument {
         url: url.to_string(),
         method: HttpMethod::GET,
@@ -150,10 +144,7 @@ fn make_get_request(
         BtcNetwork::Testnet => format!("https://{}/testnet/api/tx/{}/raw", host, txid),
         BtcNetwork::Regtest { .. } => panic!("Request to regtest network requires POST"),
     };
-    let request_headers = vec![HttpHeader {
-        name: "Host".to_string(),
-        value: format!("{host}:443"),
-    }];
+    let request_headers = vec![];
     CanisterHttpRequestArgument {
         url: url.to_string(),
         method: HttpMethod::GET,