From ca32decdadf57efb6c6eca092e7303a039fdfd37 Mon Sep 17 00:00:00 2001 From: Shu Ding Date: Mon, 15 May 2023 15:07:49 +0200 Subject: [PATCH] Fix cookies().set missing in types (#49806) Closes #49259. Note that even if we allow `.set` in the types, it will still error when called in wrong places (such as Server Components). fix NEXT-1090 --- .../server/web/spec-extension/adapters/request-cookies.ts | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/packages/next/src/server/web/spec-extension/adapters/request-cookies.ts b/packages/next/src/server/web/spec-extension/adapters/request-cookies.ts index bb49a0a9a82f..b4b0daa08ce3 100644 --- a/packages/next/src/server/web/spec-extension/adapters/request-cookies.ts +++ b/packages/next/src/server/web/spec-extension/adapters/request-cookies.ts @@ -12,7 +12,7 @@ import { ReflectAdapter } from './reflect' export class ReadonlyRequestCookiesError extends Error { constructor() { super( - 'ReadonlyRequestCookies cannot be modified. Read more: https://nextjs.org/docs/api-reference/cookies' + 'Cookies can only be modified in a Server Action or Route Handler. Read more: https://nextjs.org/docs/app/api-reference/functions/cookies#cookiessetname-value-options' ) } @@ -21,10 +21,7 @@ export class ReadonlyRequestCookiesError extends Error { } } -export type ReadonlyRequestCookies = Omit< - RequestCookies, - 'clear' | 'delete' | 'set' -> +export type ReadonlyRequestCookies = Omit export class RequestCookiesAdapter { public static seal(cookies: RequestCookies): ReadonlyRequestCookies {