From 0a5cdfc57a82452ab4f5382ed6ed5151e1ec42dd Mon Sep 17 00:00:00 2001 From: Jason Fischer Date: Wed, 14 Sep 2022 11:08:36 -0700 Subject: [PATCH] Refactor setting cookie header into own method --- src/utils/http.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/utils/http.js b/src/utils/http.js index 76f0f8b8..9f663cd4 100644 --- a/src/utils/http.js +++ b/src/utils/http.js @@ -5,13 +5,17 @@ import { Cookie, CookieJar } from 'tough-cookie'; const cookieJar = new CookieJar(); -function addCookieHandler(url, params) { +function setCookieHeader(url, params) { // add cookie header, if we have one in the jar const existingCookie = cookieJar.getCookieStringSync(url.toString()); if (existingCookie) { params.headers = params.headers ?? {}; params.headers.Cookie = existingCookie; } +} + +function addCookieHandler(url, params) { + setCookieHeader(url, params); // handle cookies during redirects params.beforeRedirect = (options, responseInfo) => { @@ -30,9 +34,7 @@ function addCookieHandler(url, params) { cookieJar.setCookieSync(cookies[i], options.href); } - const cookie = cookieJar.getCookieStringSync(options.href); - options.headers = options.headers ?? {}; - options.headers.Cookie = cookie; + setCookieHeader(options.href, options); }; }