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

Update README with ssl notes #33

Open
arndt-s opened this issue Apr 11, 2020 · 2 comments
Open

Update README with ssl notes #33

arndt-s opened this issue Apr 11, 2020 · 2 comments

Comments

@arndt-s
Copy link

arndt-s commented Apr 11, 2020

Please update the README file and add a section where ssl is documented:

I've had a problem that my configuration in config/cms.php was overwritten by CMS_LINK_POLICY env of the docker image. Same goes for APP_URL etc.

@petehalverson
Copy link
Member

Hi @arndt-s!

Agreed. I think we should add some helpful HTTPS examples.

By default, APP_ENV=docker when a container is started. The variables defined in config/docker/cms.php will override those set in config/cms.php unless otherwise overridden.

@programster
Copy link

programster commented Jun 25, 2020

An example in the README would be very useful.

It took me a while to figure out all the things I had to do. If it helps, I posted my solution (using docker with jwilder reverse proxy) here: https://blog.programster.org/deploy-october-cms-with-docker#adding-ssl

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

No branches or pull requests

3 participants