search

mautic / docker-mautic

Docker Image for Mautic
https://www.mautic.org
362 stars 274 forks source link

enable ssl connection for db in apache docker #187

Closed jbouzekri closed 3 years ago

jbouzekri commented 3 years ago

I am not sure if you are interested in merging this. In the cloud most of the managed database have SSL enabled per default so it is mandatory to be able to setup ssl options in the db connection. I chose to not change the mautic base code to add this.

cla-bot[bot] commented 3 years ago

Thank you for your contribution! We require all contributors to sign our Contributor License Agreement, and we do not have a record of your signature on file. In order for us to review and merge your code, please head over to https://www.mautic.org/contributor-agreement and complete the form. There may be a short delay while the team add you as a contributor - please be patient :). Any problems contact the Product Team on Slack (get an invite at https://mautic.org/slack). CLA has not been signed by @jbouzekri.

jbouzekri commented 3 years ago

Does not work because of the installer step. Is this step mandatory ? Can the installer be skipped as we provide database connection settings in env variable and it generates a simple app/config/local.php ?

RCheesley commented 3 years ago

@cla-bot check

cla-bot[bot] commented 3 years ago

The CLA Bot has been sent on a mission to check against the latest list and will be back shortly with its findings!

luizeof commented 3 years ago

@jbouzekri thanks for contribution ;)

please, use mautic3 branch ;)

luizeof commented 3 years ago

hello @jbouzekri

could you rebase your pull to the mautic4 branch?

jbouzekri commented 3 years ago

I am closing it as I am not using mautic anymore.