-
Notifications
You must be signed in to change notification settings - Fork 351
chore(nextjs): Updated URL for auth
cant detect usage of clerkMiddleware
#6035
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🦋 Changeset detectedLatest commit: b2d128a 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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/elements
@clerk/clerk-expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/clerk-react
@clerk/react-router
@clerk/remix
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/themes
@clerk/types
@clerk/upgrade
@clerk/vue
commit: |
auth
can't detect usage of clerkMiddleware
auth
can't detect usage of clerkMiddleware
auth
cant detect usage of clerkMiddleware
Thanks for fixing the title/linting issue @tmilewski! Sorry I didn't get to it first. I marked this blocked to make sure it doesn't get merged unless the Docs PR gets merged too. Appreciate the reviews Tom and @wobsoriano! |
Love the idea of pointing this error to a dedicated troubleshooting page—it's a DX win for users! I do have one long‑term concern, though, which is link durability: once we ship a hard‑coded URL in the package, any future re‑org of docs risks breaking it. I think it would be worthwhile to add a simple shortcode/redirect layer (e.g., https://clerk.com/err/auth-middleware) that resolves to the current canonical doc? Benefits:
I think this would be very low hanging fruit but could give us a ton of flexibility and visibility into SDK error messages |
I think this is a great idea! Thanks for making it. I'm going to share with the docs team. The I updated to |
Description
This PR goes hand in hand with Docs PR 2273
The @clerk/nextjs error "Clerk: auth() was called but Clerk can't detect usage of clerkMiddleware()" is confusing to customers, and it would be hard to cover explaining links to missing static assets and fixing that in in the error.
This borrows from Next.js. The Docs PR includes a entire page covering the error and how to solve the reasons it can occur. This PR changes the link in the error from the Quickstart to the new page.
Checklist
pnpm test
runs as expected.pnpm build
runs as expected.Type of change