From 0a8dacda24216640e7318830bd8bd58a8d564c71 Mon Sep 17 00:00:00 2001
From: Sander Declerck <sander@sdsoftware.be>
Date: Mon, 24 Nov 2025 15:25:56 +0100
Subject: [PATCH] Add small comment on why we're removing the host header
 before forwarding.

---
 packages/safe-chain/src/registryProxy/mitmRequestHandler.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/safe-chain/src/registryProxy/mitmRequestHandler.js b/packages/safe-chain/src/registryProxy/mitmRequestHandler.js
index c4e7bb7..bfc6c3e 100644
--- a/packages/safe-chain/src/registryProxy/mitmRequestHandler.js
+++ b/packages/safe-chain/src/registryProxy/mitmRequestHandler.js
@@ -151,6 +151,8 @@ function forwardRequest(req, hostname, res, requestHandler) {
 function createProxyRequest(hostname, req, res, requestHandler) {
   /** @type {NodeJS.Dict<string | string[]> | undefined} */
   let headers = { ...req.headers };
+  // Remove the host header from the incoming request before forwarding.
+  // Node's http module sets the correct host header for the target hostname automatically.
   if (headers.host) {
     delete headers.host;
   }