search

aaronpk / indielogin.com

Sign in with your domain name
https://indielogin.com
MIT License
150 stars 24 forks source link

Docker support #34

Open filips123 opened 5 years ago

filips123 commented 5 years ago

I've added basic support for Docker. It is part of the fix for #27. That issue should still be opened because it is still not completed.

It is now possible to run the website with docker-compose up. It contains several containers for PHP, Ruby, SQL and Redis.

After cloning the repo, users should copy lib/config.docker.php to lib/config.php (and not edit it). That file loads config from environment variables.

Then, they should copy .env.sample to .env and provide configuration.

They can then start all containers with docker-compose up.

This has some problems.

The database is empty. Even if table schema is added at first run, tables will be empty. And this is a problem because the client_id would not be registered. There should be some admin website or console command to add clients.

Also, documentation is not finished. It needs to be finished after all other problems are fixed.

And all functionality needs to be tested because I didn't have time to do this completely.

Lewiscowles1986 commented 5 years ago

I've made a PR which I think explains what I think is a better approach to ENV based configuration. It was what I was trying to explain to you.

Basically there should be no need for Config in environments such as Docker