Skip to content

fix: use bcryptjs instead of bcrypt to avoid native module errors on …#1040

Open
vdzhagaev wants to merge 1 commit intovercel:mainfrom
vdzhagaev:fix/vercel-build-bcrypt
Open

fix: use bcryptjs instead of bcrypt to avoid native module errors on …#1040
vdzhagaev wants to merge 1 commit intovercel:mainfrom
vdzhagaev:fix/vercel-build-bcrypt

Conversation

@vdzhagaev
Copy link
Copy Markdown

This PR replaces the bcrypt package with bcryptjs to fix build errors on Vercel caused by missing native bindings.

Changes:

  • Removed bcrypt, added bcryptjs
  • Updated import in app/seed/route.ts
  • Updated pnpm-lock.yaml accordingly

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-learn-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 7:44pm
next-seo-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 7:44pm

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 15, 2025

@bee4boop is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedbcryptjs@​3.0.21001001009090

View full report

@XingxinHE
Copy link
Copy Markdown

I think this might be due to different environment? See the troubleshooting at https://nextjs.org/learn/dashboard-app/setting-up-your-database

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants