Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix docker development image, add env validation and small fixes #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dragonhunt02
Copy link
Contributor

@dragonhunt02 dragonhunt02 commented Feb 16, 2025

General fixes

  • Fixed config/prod.exs not loading (MIX_ENV=prod was not set)
  • Interpolate variables in .env files for easier configuration
  • Move env variable validation after application startup, since .env variables are not available at build time in Docker.
  • Update Quick Setup instructions

Development image fixes

  • Remove frontend volume mount that triggered build failure. frontend/ folder in docker image was replaced by docker-compose when mounting, so node modules installed in image were removed.
  • Removed frontend entrypoint override that caused failure on image run
  • Fix missing depends_on, links causing build failure. In docker-compose files, extends won't extend some properties (see Restrictions section)
  • Add development Caddyfile to enable HTTP without TLS server on port 80 for testing
  • /mailbox load failed with a 401 error. As a temporary fix it was moved to /api/v1/mailbox path, but this issue needs review.

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