Fixed app/templates/new_domain.j2

[narate]

Fix invalid Nginx config template and change access_log to /tmp/{{ name }}.access.log to avoid file permission.

from

{
    listen          80;
    server_name     {{ name }},
    access_log      logs/{{ name }}.access.log main;

    location / {
        proxy_pass      http://127.0.0.1:8080;
    }

}

to

server {
    listen          80;
    server_name     {{ name }};
    access_log      /tmp/{{ name }}.access.log main;

    location / {
        proxy_pass  http://127.0.0.1:8080;
    }
}

[LucaNerlich]

why do you want to store the logs in /tmp/?

[narate]

The default user for run nginx in nginx docker image is nginx, but logs/* permission is for root:root.

Then if we run nginx -t will get error

nginx -t
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
2020/06/25 20:06:24 [emerg] 25#25: open() "/etc/nginx/logs/test.com.access.log" failed (2: No such file or directory)
nginx: [emerg] open() "/etc/nginx/logs/test.com.access.log" failed (2: No such file or directory)
nginx: configuration file /etc/nginx/nginx.conf test failed

[narate]

I think access_log /dev/stdout main; is fine for docker container

[schenkd]

Hello @narate, thank you very much for your contribution. The default template is only a minimal template. When creating a new domain, everyone has the possibility to customize it. This will be fundamentally revised in the next release anyway. For your problem with the access log I see the change of the path as a workaround and no solution for the permission problem. Therefore I have to decline this PR.