Skip to content

Conversation

@claudeaceae
Copy link

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.yaml file uses BETTER_AUTH_URL, but the actual codebase expects NEXT_PUBLIC_BETTER_AUTH_URL.

Evidence:

  • packages/auth/src/server.ts (lines 8 and 45) reads NEXT_PUBLIC_BETTER_AUTH_URL
  • apps/web/.env.example uses NEXT_PUBLIC_BETTER_AUTH_URL
  • All documentation (README.md, CONTRIBUTING.md) specifies NEXT_PUBLIC_BETTER_AUTH_URL

Changes Made

Changed line 62 in docker-compose.yaml:

- BETTER_AUTH_URL=http://localhost:3000
+ NEXT_PUBLIC_BETTER_AUTH_URL=http://localhost:3000

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:

  1. Checking packages/auth/src/server.ts - it destructures NEXT_PUBLIC_BETTER_AUTH_URL from keys()
  2. Checking the Better Auth configuration - it uses baseURL: NEXT_PUBLIC_BETTER_AUTH_URL

Note: This is a configuration fix that aligns docker-compose.yaml with the actual codebase requirements.

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.
@vercel
Copy link

vercel bot commented Dec 22, 2025

@claudeaceae is attempting to deploy a commit to the OpenCut OSS Team on Vercel.

A member of the Team first needs to authorize it.

@netlify
Copy link

netlify bot commented Dec 22, 2025

👷 Deploy request for appcut pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit e066376

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 22, 2025

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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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.

📥 Commits

Reviewing files that changed from the base of the PR and between 0173db9 and e066376.

📒 Files selected for processing (1)
  • docker-compose.yaml
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant