Skip to content

Commit 1358ccc

Browse files
kkarikosdummdidummbenmccann
authored
fix: use default cookie decoder (#13037)
Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com>
1 parent 239fe18 commit 1358ccc

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

Diff for: .changeset/plenty-oranges-count.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@sveltejs/kit': patch
3+
---
4+
5+
fix: do not override default cookie decoder to allow users to override the `cookie` library version

Diff for: packages/kit/src/runtime/server/cookie.js

+2-4
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,7 @@ export function get_cookies(request, url, trailing_slash) {
6767
return c.value;
6868
}
6969

70-
const decoder = opts?.decode || decodeURIComponent;
71-
const req_cookies = parse(header, { decode: decoder });
70+
const req_cookies = parse(header, { decode: opts?.decode });
7271
const cookie = req_cookies[name]; // the decoded string or undefined
7372

7473
// in development, if the cookie was set during this session with `cookies.set`,
@@ -95,8 +94,7 @@ export function get_cookies(request, url, trailing_slash) {
9594
* @param {import('cookie').CookieParseOptions} opts
9695
*/
9796
getAll(opts) {
98-
const decoder = opts?.decode || decodeURIComponent;
99-
const cookies = parse(header, { decode: decoder });
97+
const cookies = parse(header, { decode: opts?.decode });
10098

10199
for (const c of Object.values(new_cookies)) {
102100
if (

0 commit comments

Comments
 (0)