Skip to content
This repository was archived by the owner on Jul 30, 2023. It is now read-only.

Commit 53ccbee

Browse files
committed
fix(MergeRequestHeaders - add reduce initial value)
1 parent 379b860 commit 53ccbee

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

cookie.js

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cookie.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ function resolveCookieString(
2121
const searchReg = existing ? /^cookie$/i : /^set-cookie$/i;
2222
const key = Object.keys(headers).find(key => Boolean(key.match(searchReg)));
2323
const cookie = headers[key];
24-
return Array.isArray(cookie) ? cookie.reduce((s1, s2) => `${s1},${s2}`) : cookie;
24+
return Array.isArray(cookie) ? cookie.reduce((s1, s2) => `${s1},${s2}`, '') : cookie;
2525
}
2626

2727
export function handleCookie(url: string, headers: Headers) {
@@ -52,7 +52,9 @@ export function mergeRequestHeaders(url: string, headers = {}): { [key: string]:
5252
console.log(error.message);
5353
}
5454

55-
const mergedCookie = [cookies, existingCookies].filter(str => str).reduce((s1, s2) => s1 + s2);
55+
const mergedCookie = [cookies, existingCookies]
56+
.filter(str => str)
57+
.reduce((s1, s2) => s1 + s2, '');
5658

5759
return mergedCookie ? { ...headers, Cookie: mergedCookie } : headers;
5860
}

0 commit comments

Comments
 (0)