search

standardnotes / syncing-server

[Deprecated: Use our new Node server: https://github.com/standardnotes/syncing-server-js]
https://standardnotes.org
GNU Affero General Public License v3.0
209 stars 48 forks source link

Bug: mysql environment variables are not populated in docker-compose.yml #171

Closed dgraziotin closed 3 years ago

dgraziotin commented 3 years ago

These environment variables:

environment:
      MYSQL_DATABASE: '${DB_DATABASE}'
      MYSQL_USER: '${DB_USERNAME}'
      MYSQL_PASSWORD: '${DB_PASSWORD}'
      MYSQL_ROOT_PASSWORD: '${DB_PASSWORD}'

are populated only when adding env_file: .envbefore them. Otherwise they are empty and the database will not be accessed.

dgraziotin commented 3 years ago

Actually, not. Sorry! But the current docker-compose.yml is broken.

JaspalSuri commented 3 years ago

Hi @dgraziotin,

But the current docker-compose.yml is broken.

Can you elaborate a bit more on this and when it fails to work as expected?

dgraziotin commented 3 years ago

Hi @JaspalSuri, actually my bad. Lots of permission issues when building/running this as root. Switching to a non-root user makes this run perfectly fine!

JaspalSuri commented 3 years ago

I'm glad to hear that! Thanks for letting us know. 😃