fix(infra): Prisma engine crashes due to OpenSSL version mismatch#200
fix(infra): Prisma engine crashes due to OpenSSL version mismatch#200
Conversation
…event OpenSSL version mismatch on Node 22+/OpenSSL 3.5+
…agement - Updated middleware to allow access to login pages when ?force=true is present, preventing redirect loops for authenticated users. - Improved cookie clearing logic in auth context to ensure compatibility across browsers by setting both max-age and expires attributes. - Refactored RequireAuth component to always clear auth storage before redirecting, addressing potential stale session issues. These changes improve the robustness of the authentication flow and enhance user experience during login attempts.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
🗃️ Database Migration Detected This PR includes changes to Prisma schema files. Please ensure:
Requesting review from the core team: @livepeer/core |
📝 WalkthroughWalkthroughUpdates Node.js version requirement to 20, simplifies RequireAuth component by removing unused auth context values from dependency tracking, and adjusts Prisma generator configuration to prevent OpenSSL compatibility issues. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
⏳ Review reminder — This PR has been waiting for review for over 24 hours. @team — this PR needs a review. |
|
🚨 Review SLA exceeded (72 h) — This PR has been waiting for review for over 72 hours. Please prioritize. @team — urgent review needed. |
Summary
Removes the
"native"binary target from the Prisma schema to prevent engine crashes caused by OpenSSL version mismatches on Node 22+ / OpenSSL 3.5+ environments. Pins the project to Node 20 LTS via.nvmrcto lock dev environments to a known-stable OpenSSL 3.0.x combination.Changes
"native"frombinaryTargetsinpackages/database/prisma/schema.prisma— all required deployment targets are already explicitly listed (darwin-arm64,linux-arm64-openssl-3.0.x,debian-openssl-3.0.x,rhel-openssl-3.0.x,linux-musl-openssl-3.0.x), making"native"redundant and a source of mismatches.nvmrcpinning Node 20 LTS to prevent accidental local regressions on Node 22+Type
Plugin(s) Affected
Checklist
npm run lint)npm run build)Breaking Changes
None — removing
"native"does not affect any deployment target. All production platforms are covered by the explicit targets that remain.Screenshots / Recordings