clerk / clerk/javascript

bug(nuxt): repeated Set-Cookie headers are overwritten by clerkMiddleware

Open Beginner friendly
#9,573 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
1.8k
Forks
472
Avg merge
2d 11h
Merged PRs (30d)
189

Description

Preliminary Checks
Reproduction

https://github.com/BalajiSriraman/javascript/blob/fix/nuxt-preserve-set-cookie-headers/packages/nuxt/src/runtime/server/__tests__/clerkMiddleware.test.ts#L116-L143

The package's existing clerkMiddleware.test.ts can reproduce this without a Clerk instance: mock authenticateRequest() with a Headers object containing two appended Set-Cookie values, run the H3 handler, then inspect response.headers.getSetCookie(). Only the last value remains.

Publishable key

Not applicable. This is a unit-level server middleware reproduction and does not make a request to Clerk.

Description

Steps to reproduce:

  1. Create a Headers instance and append two Set-Cookie values, such as an expired __clerk_handshake cookie and a refreshed __session cookie.
  2. Return those headers from the mocked authenticateRequest() result used by clerkMiddleware.
  3. Send a request through the H3 handler and inspect response.headers.getSetCookie().

Expected behavior:

Both cookie directives are present in the Nuxt response. Set-Cookie is a repeatable response header, and Clerk can return several cookie mutations from one authentication or handshake result.

Actual behavior:

Only the last cookie directive is present. clerkMiddleware iterates over the authentication headers and calls H3's setResponseHeader() for every value. That API replaces a previous value with the same name, so each Set-Cookie overwrites the preceding one.

This can leave an expired or stale Clerk handshake cookie in the browser when its deletion directive is followed by another cookie directive. Subsequent server requests can then continue entering the handshake path until that short-lived cookie expires.

Environment
System:
  OS: Linux 6.16 Pop!_OS 24.04 LTS
  CPU: (16) x64 11th Gen Intel(R) Core(TM) i9-11900K @ 3.50GHz
Binaries:
  Node: 25.0.0
  npm: 11.6.2
  pnpm: 10.33.0
Browsers:
  Chrome: 150.0.7871.46
npmPackages:
  @clerk/nuxt: 3.0.15
  h3: 1.15.11

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with packages/nuxt/src/runtime/server/tests/clerkMiddleware.test.ts, especially the reproduction around lines 116-143, and inspect the clerkMiddleware path exercised by the H3 handler. Run the focused test with mocked authentication headers; done means both appended Set-Cookie directives remain available in the Nuxt response.

Written by the indexing model from the issue text.

Assessment

Tech stack
nuxtjs, typescript
Domain
authentication, backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
86/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.