Skip to content

Conversation

@anagstef
Copy link
Member

@anagstef anagstef commented Sep 24, 2024

Description

This PR is dropping the __clerk_refresh debugging query param and uses the __clerk_hs_reason param for all the different handshake scenarios.

All errors are the following:

  • 'client-uat-but-no-session-token'
  • 'dev-browser-missing'
  • 'dev-browser-sync'
  • 'primary-responds-to-syncing'
  • 'satellite-needs-syncing'
  • 'session-token-and-uat-missing'
  • 'session-token-iat-before-client-uat'
  • 'session-token-nbf'
  • 'session-token-iat-in-the-future'
  • 'session-token-but-no-client-uat'
  • 'session-token-expired-refresh-non-eligible-no-refresh-cookie'
  • 'session-token-expired-refresh-non-eligible-non-get'
  • 'session-token-expired-refresh-invalid-session-token'
  • 'session-token-expired-refresh-missing-api-client'
  • 'session-token-expired-refresh-missing-session-token'
  • 'session-token-expired-refresh-missing-refresh-token'
  • 'session-token-expired-refresh-expired-session-token-decode-failed'
  • 'session-token-expired-refresh-fetch-error'
  • 'session-token-expired-refresh-unexpected-sdk-error'
  • 'session-token-expired-refresh-<bapi_error_code>'

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:

…e only the `__clerk_hs_reason` param for all scenarios
@changeset-bot
Copy link

changeset-bot bot commented Sep 24, 2024

🦋 Changeset detected

Latest commit: 067abaf

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

This PR includes changesets to release 9 packages
Name Type
@clerk/backend Patch
@clerk/astro Patch
@clerk/express Patch
@clerk/fastify Patch
@clerk/nextjs Patch
@clerk/remix Patch
@clerk/clerk-sdk-node Patch
@clerk/tanstack-start Patch
@clerk/testing 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

@anagstef
Copy link
Member Author

!snapshot

@anagstef anagstef changed the title fix(backend): Drop the __clerk_refresh debugging query param fix(backend): Drop the __clerk_refresh debugging query param Sep 24, 2024
@clerk-cookie
Copy link
Collaborator

Hey @anagstef - the snapshot version command generated the following package versions:

Package Version
@clerk/astro 1.3.7-snapshot.vf8c993f
@clerk/backend 1.13.3-snapshot.vf8c993f
@clerk/chrome-extension 1.3.9-snapshot.vf8c993f
@clerk/clerk-js 5.22.4-snapshot.vf8c993f
@clerk/elements 0.15.5-snapshot.vf8c993f
@clerk/clerk-expo 2.2.15-snapshot.vf8c993f
@clerk/express 1.0.0-snapshot.vf8c993f
@clerk/fastify 1.0.46-snapshot.vf8c993f
@clerk/localizations 3.0.5-snapshot.vf8c993f
@clerk/nextjs 5.6.1-snapshot.vf8c993f
@clerk/clerk-react 5.9.2-snapshot.vf8c993f
@clerk/remix 4.2.30-snapshot.vf8c993f
@clerk/clerk-sdk-node 5.0.43-snapshot.vf8c993f
@clerk/shared 2.8.2-snapshot.vf8c993f
@clerk/tanstack-start 0.4.6-snapshot.vf8c993f
@clerk/testing 1.3.4-snapshot.vf8c993f
@clerk/themes 2.1.31-snapshot.vf8c993f
@clerk/types 4.21.1-snapshot.vf8c993f

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/astro

npm i @clerk/astro@1.3.7-snapshot.vf8c993f --save-exact

@clerk/backend

npm i @clerk/backend@1.13.3-snapshot.vf8c993f --save-exact

@clerk/chrome-extension

npm i @clerk/chrome-extension@1.3.9-snapshot.vf8c993f --save-exact

@clerk/clerk-js

npm i @clerk/clerk-js@5.22.4-snapshot.vf8c993f --save-exact

@clerk/elements

npm i @clerk/elements@0.15.5-snapshot.vf8c993f --save-exact

@clerk/clerk-expo

npm i @clerk/clerk-expo@2.2.15-snapshot.vf8c993f --save-exact

@clerk/express

npm i @clerk/express@1.0.0-snapshot.vf8c993f --save-exact

@clerk/fastify

npm i @clerk/fastify@1.0.46-snapshot.vf8c993f --save-exact

@clerk/localizations

npm i @clerk/localizations@3.0.5-snapshot.vf8c993f --save-exact

@clerk/nextjs

npm i @clerk/nextjs@5.6.1-snapshot.vf8c993f --save-exact

@clerk/clerk-react

npm i @clerk/clerk-react@5.9.2-snapshot.vf8c993f --save-exact

@clerk/remix

npm i @clerk/remix@4.2.30-snapshot.vf8c993f --save-exact

@clerk/clerk-sdk-node

npm i @clerk/clerk-sdk-node@5.0.43-snapshot.vf8c993f --save-exact

@clerk/shared

npm i @clerk/shared@2.8.2-snapshot.vf8c993f --save-exact

@clerk/tanstack-start

npm i @clerk/tanstack-start@0.4.6-snapshot.vf8c993f --save-exact

@clerk/testing

npm i @clerk/testing@1.3.4-snapshot.vf8c993f --save-exact

@clerk/themes

npm i @clerk/themes@2.1.31-snapshot.vf8c993f --save-exact

@clerk/types

npm i @clerk/types@4.21.1-snapshot.vf8c993f --save-exact

@nikosdouvlis nikosdouvlis merged commit 5c18671 into main Sep 24, 2024
@nikosdouvlis nikosdouvlis deleted the stefanos/drop-clerk-refresh-query-param branch September 24, 2024 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants