fix: correct Better Auth URL variable in docker-compose.yaml#668
fix: correct Better Auth URL variable in docker-compose.yaml#668claudeaceae wants to merge 1 commit intoOpenCut-app:mainfrom
Conversation
The docker-compose.yaml was using BETTER_AUTH_URL, but the actual codebase expects NEXT_PUBLIC_BETTER_AUTH_URL. This is evident in: - packages/auth/src/server.ts (line 8, 45) - apps/web/.env.example This fix ensures Docker deployments use the correct environment variable name, preventing authentication configuration errors.
|
@claudeaceae is attempting to deploy a commit to the OpenCut OSS Team on Vercel. A member of the Team first needs to authorize it. |
👷 Deploy request for appcut pending review.Visit the deploys page to approve it
|
|
Warning Rate limit exceeded@claudeaceae has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 27 minutes and 13 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ 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 |
Summary
This PR fixes an environment variable name mismatch in docker-compose.yaml that would cause authentication to fail in Docker deployments.
The Problem
The
docker-compose.yamlfile usesBETTER_AUTH_URL, but the actual codebase expectsNEXT_PUBLIC_BETTER_AUTH_URL.Evidence:
packages/auth/src/server.ts(lines 8 and 45) readsNEXT_PUBLIC_BETTER_AUTH_URLapps/web/.env.exampleusesNEXT_PUBLIC_BETTER_AUTH_URLNEXT_PUBLIC_BETTER_AUTH_URLChanges Made
Changed line 62 in docker-compose.yaml:
Impact
Without this fix, users deploying with Docker would encounter authentication failures because the Better Auth library wouldn't be able to find its base URL configuration.
Testing
This can be verified by:
packages/auth/src/server.ts- it destructuresNEXT_PUBLIC_BETTER_AUTH_URLfrom keys()baseURL: NEXT_PUBLIC_BETTER_AUTH_URLNote: This is a configuration fix that aligns docker-compose.yaml with the actual codebase requirements.