From b299879b82ce90de3562052d877c6bd4c7584cac Mon Sep 17 00:00:00 2001
From: sanjai0py <sanjailucifer666@gmail.com>
Date: Wed, 21 May 2025 17:00:22 +0530
Subject: [PATCH] Refactor saveCookies function to remove disableCookies
 parameter and streamline cookie handling in response interceptors

---
 packages/bruno-cli/src/utils/axios-instance.js | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/packages/bruno-cli/src/utils/axios-instance.js b/packages/bruno-cli/src/utils/axios-instance.js
index ed2e67f1b..d3fea8f6d 100644
--- a/packages/bruno-cli/src/utils/axios-instance.js
+++ b/packages/bruno-cli/src/utils/axios-instance.js
@@ -5,10 +5,7 @@ const { addCookieToJar, getCookieStringForUrl } = require('./cookies');
 const redirectResponseCodes = [301, 302, 303, 307, 308];
 const METHOD_CHANGING_REDIRECTS = [301, 302, 303];
 
-const saveCookies = (url, headers, disableCookies) => {
-  if (disableCookies) {
-    return;
-  }
+const saveCookies = (url, headers) => {
   if (headers['set-cookie']) {
     let setCookieHeaders = Array.isArray(headers['set-cookie'])
       ? headers['set-cookie']
@@ -85,7 +82,6 @@ function makeAxiosInstance({ requestMaxRedirects = 5, disableCookies } = {}) {
       response.headers['request-duration'] = end - start;
       redirectCount = 0;
 
-      saveCookies(response.config.url, response.headers, disableCookies);
       return response;
     },
     (error) => {
@@ -114,7 +110,10 @@ function makeAxiosInstance({ requestMaxRedirects = 5, disableCookies } = {}) {
             redirectUrl = URL.resolve(error.config.url, locationHeader);
           }
 
-          saveCookies(redirectUrl, error.response.headers, disableCookies);
+          if (!disableCookies){
+            saveCookies(redirectUrl, error.response.headers);
+          }
+
           const requestConfig = createRedirectConfig(error, redirectUrl);
 
           if (!disableCookies) {