dokku / ansible-dokku

Ansible modules for installing and configuring Dokku
MIT License
176 stars 44 forks source link

Nginx management happens when dokku_manage_nginx is false #200

Open robotski opened 2 months ago

robotski commented 2 months ago

I think the when is misaligned here, currently it applies only to nginx:enable default instead of the whole block, which seems like incorrect behavior to me.

https://github.com/dokku/ansible-dokku/blob/ca2f58681a3dcc062066bd291d4cc65d8472cb68/tasks/nginx.yml#L44

josegonzalez commented 1 month ago

Pull requests welcome!