Skip to content

fix(clerk-js): Fix same site in ClerkJS cookies#2694

Merged
nikosdouvlis merged 2 commits into
release/v4from
fix_cookies_same_site
Feb 1, 2024
Merged

fix(clerk-js): Fix same site in ClerkJS cookies#2694
nikosdouvlis merged 2 commits into
release/v4from
fix_cookies_same_site

Conversation

@SokratisVidros

Copy link
Copy Markdown
Contributor

Description

__client_uat is samesite=none
__session is samesite=lax
__clerk_db_jwt is samesite=lax
__initted is samesite=lax

Fixes #2668

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Packages affected

  • @clerk/backend
  • @clerk/chrome-extension
  • @clerk/clerk-js
  • @clerk/clerk-expo
  • @clerk/fastify
  • gatsby-plugin-clerk
  • @clerk/localizations
  • @clerk/nextjs
  • @clerk/clerk-react
  • @clerk/remix
  • @clerk/clerk-sdk-node
  • @clerk/shared
  • @clerk/themes
  • @clerk/types
  • build/tooling/chore

__client_uat is samesite=none
__session is samesite=lax
__clerk_db_jwt is samesite=lax
__initted is samesite=lax

Fixes #2668
@SokratisVidros SokratisVidros self-assigned this Jan 31, 2024
@SokratisVidros SokratisVidros requested a review from a team as a code owner January 31, 2024 14:45
@SokratisVidros SokratisVidros requested review from anagstef and removed request for a team January 31, 2024 14:45
@changeset-bot

changeset-bot Bot commented Jan 31, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4c5ee86

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@clerk/clerk-js Patch
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants