From be903049ed8ae4cc6ddfd0e096558dac08b35555 Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Tue, 14 Mar 2023 23:47:51 +0100
Subject: [PATCH] fix(backend): whitelist for include_header was ignored in
 some cases

---
 backend/windmill-api/src/jobs.rs | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/backend/windmill-api/src/jobs.rs b/backend/windmill-api/src/jobs.rs
index b26868d46a..8b84fd4d88 100644
--- a/backend/windmill-api/src/jobs.rs
+++ b/backend/windmill-api/src/jobs.rs
@@ -300,9 +300,6 @@ pub fn add_include_headers(
     headers: HeaderMap,
     mut args: serde_json::Map<String, serde_json::Value>,
 ) -> serde_json::Map<String, serde_json::Value> {
-    if include_header.is_none() {
-        return args;
-    }
     let whitelist = include_header
         .as_ref()
         .map(|s| s.split(",").map(|s| s.to_string()).collect::<Vec<_>>())
@@ -321,6 +318,7 @@ pub fn add_include_headers(
         });
     args
 }
+
 #[derive(Deserialize)]
 pub struct ListQueueQuery {
     pub script_path_start: Option<String>,