feat(remix): Support handshake flow for remix#2380
Conversation
🦋 Changeset detectedLatest commit: 6e4aa1b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
nikosdouvlis
left a comment
There was a problem hiding this comment.
This is pretty much all that's needed :)
We might need to do something in ssr/getAuth as well. I did replace the error thrown in a previous PR with a redirect('') but could you please check whether we need something more or not? (and remove the todo ofc)
We want to halt execution in the nested loaders as all loaders (nested and root) run in parallel - we halt execution by throwing a redirect or an error. Remix evaluates errors first, so we should test that calling redirect with '' does indeed throw a redirect and not an error because of the invalid URL
| '@clerk/remix': minor | ||
| --- | ||
|
|
||
| Update the Remix rootAuthLoader to handle handshake auth status, this replaces the previous interstitial flow. |
There was a problem hiding this comment.
Let's include a short migration guide as the ClerkErrorBoundary export is no longer needed :)
…into authenticateRequest
Description
Support the handshake flow for Remix.
Checklist
npm testruns as expected.npm run buildruns as expected.Type of change
Packages affected
@clerk/backend@clerk/chrome-extension@clerk/clerk-js@clerk/clerk-expo@clerk/fastifygatsby-plugin-clerk@clerk/localizations@clerk/nextjs@clerk/clerk-react@clerk/remix@clerk/clerk-sdk-node@clerk/shared@clerk/themes@clerk/typesbuild/tooling/chore