fix: remove default env vars in docker compose

[ShubhamPalriwala]

What does this PR do?

Removes default environment variables in docker-compose.yml

Checklist

Required

• Filled out the "How to test" section in this PR
• Read How we Code at Formbricks
• Self-reviewed my own code
• Commented on my code in hard-to-understand bits
• Ran pnpm build
• Checked for warnings, there are none
• Removed all console.logs
• Merged the latest changes from main onto my branch with git pull origin main
• My changes don't cause any responsiveness issues

Appreciated

• If a UI change was made: Added a screen recording or screenshots to this PR
• Updated the Formbricks Docs if changes were necessary

[vercel]

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

2 Ignored Deployments

Name	Status	Preview	Updated (UTC)
formbricks-cloud	⬜️ Ignored (Inspect)		Feb 5, 2024 3:57pm
formbricks-com	⬜️ Ignored (Inspect)		Feb 5, 2024 3:57pm

[github-actions]

Thank you for following the naming conventions for pull request titles! 🙏

[review-agent-prime]

docker-compose.yml

It's a good practice to avoid hardcoding sensitive information such as database URLs, secrets, encryption keys, etc., directly in the docker-compose file. Instead, these should be stored in environment variables or secure secret management systems. This change is a step in the right direction. However, to further improve, you can use Docker secrets or environment variables for managing these sensitive data.
Create Issue
See the diff
Checkout the fix

    x-database-url: &database_url ${DATABASE_URL}
    x-nextauth-secret: &nextauth_secret ${NEXTAUTH_SECRET}
    x-encryption-key: &encryption_key ${ENCRYPTION_KEY}
    x-cron-secret: &cron_secret ${CRON_SECRET}

git fetch origin && git checkout -b ReviewBot/Impro-d0bsqn7 origin/ReviewBot/Impro-d0bsqn7

[mattinannt]

@ShubhamPalriwala thank you very much for the quick turnaround! 💪