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
fix(backend,clerk-sdk-node): Fix noisy deprecations sdk node #1856
Conversation
🦋 Changeset detectedLatest commit: 3bb935d The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
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 |
d068676
to
ca60398
Compare
ca60398
to
3bb935d
Compare
@@ -26,7 +26,7 @@ export const loadClientEnv = () => { | |||
export const loadApiEnv = () => { | |||
return { | |||
secretKey: process.env.CLERK_SECRET_KEY || process.env.CLERK_API_KEY || '', | |||
apiKey: process.env.CLERK_SECRET_KEY || process.env.CLERK_API_KEY || '', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there usages of apiKey
that assume it has been defaulted to CLERK_SECRET_KEY
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it's a typo or an attempt to be "extra" safe that the env is used.
Description
Avoid always showing
frontendApi
andapiKey
on@clerk/clerk-sdk-node
!Checklist
npm test
runs as expected.npm run build
runs as expected.Type of change
Packages affected
@clerk/clerk-js
@clerk/clerk-react
@clerk/nextjs
@clerk/remix
@clerk/types
@clerk/themes
@clerk/localizations
@clerk/clerk-expo
@clerk/backend
@clerk/clerk-sdk-node
@clerk/shared
@clerk/fastify
@clerk/chrome-extension
gatsby-plugin-clerk
build/tooling/chore