Merge pull request #24 from secondlife/signal/no-access-logs

Add NO_ACCESS_LOGS option

Author: bennettgoble

Dockerfile

@@ -13,6 +13,7 @@ ENV PROXY_UWSGI=0
 ENV LISTEN_PORT=80
 ENV HEALTHCHECK_PATH="/lb-status/"
 ENV STATIC_LOCATIONS=
+ENV NO_ACCESS_LOGS=0
 EXPOSE 80
 STOPSIGNAL SIGQUIT
 ENTRYPOINT ["/docker-entrypoint.sh"]

README.md

@@ -25,6 +25,7 @@ Pair nginx-proxy with your favorite upstream server (wsgi, uwsgi, asgi, et al.)
 | `PROXY_UWSGI` | Whether to use native uwsgi support | No | 0 | 1 |
 | `KEEPALIVE_TIMEOUT` | What value to set HTTP keepalive (This should be higher than your ELB's timeout) | Yes | 65 | |
 | `HEALTHCHECK_PATH` | nginx-proxy disables healthcheck path access logs, you can configure the path here | Yes | /lb-status/ | |
+| `NO_ACCESS_LOGS` | disable access logs completely | No | 0 | 1 |
 
 ### Hosting Static Assets
 

src/etc/nginx/conf.d/default.conf.template

@@ -30,6 +30,9 @@ server {
     {{ end }}
   }
 
+  {{ if (eq .Env.NO_ACCESS_LOGS "1" ) }}
+  access_log off;
+  {{ else }}
   location {{ .Env.HEALTHCHECK_PATH }} {
     access_log off;
     {{ if (eq .Env.PROXY_UWSGI "1") }}
@@ -38,6 +41,7 @@ server {
     include /etc/nginx/includes/proxy.conf;
     {{ end }}
   }
+  {{ end }}
 
   {{ if .Env.STATIC_LOCATIONS }}
   {{ range (.Env.STATIC_LOCATIONS | strings.Split "," )}}