Allow setting access_log to "off" (#11)

Author: StudioMaX

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.x (Unreleased)
+
+ENHANCEMENTS:
+
+*   Improve configuration templating capabilities:
+    *   Allow setting `access_log`/`access_log_location` to `off`.
+
 ## 0.1.0 (August 19, 2020)
 
 Initial release of the NGINX Config role. Contains all NGINX Config related features previously available on the [NGINX Ansible role](https://github.com/nginxinc/ansible-role-nginx).

defaults/main/template.yml

@@ -94,6 +94,7 @@ nginx_config_http_template:
         include_files: []
         http_error_pages: {}
         error_page: /usr/share/nginx/html
+        # access_log: "off" # Cancels all access_log directives on the current level
         access_log:
           - name: main
             location: /var/log/nginx/access.log

molecule/common/playbooks/default_converge.yml

@@ -76,6 +76,9 @@
                 http_error_pages:
                   404: /404.html
                 error_page: /usr/share/nginx/html
+                access_log:
+                  - name: main
+                    location: /var/log/nginx/access.log
                 client_max_body_size: 512k
                 proxy_hide_headers:
                   - X-Powered-By
@@ -254,6 +257,7 @@
                     opts: []
                 server_name: localhost
                 error_page: /usr/share/nginx/html
+                access_log: "off"
                 autoindex: false
                 sub_filter:
                   sub_filters:

templates/http/default.conf.j2

@@ -491,10 +491,14 @@ server {
 {% endif %}
 
 {% if item.value.servers[server].access_log is defined %}
+{% if item.value.servers[server].access_log is sameas false or item.value.servers[server].access_log == "off" %}
+    access_log off;
+{% else %}
 {% for access_log in item.value.servers[server].access_log %}
     access_log  {{ access_log.location }}  {{ access_log.name }};
 {% endfor %}
 {% endif %}
+{% endif %}
 {% if item.value.servers[server].error_log is defined %}
     error_log {{ item.value.servers[server].error_log.location }} {{ item.value.servers[server].error_log.level }};
 {% endif %}

templates/nginx.conf.j2

@@ -66,9 +66,15 @@ http {
     log_format  {{ access_log.name }}  {{ access_log.format }};
 {% endfor %}
 
+{% if nginx_config_main_template.http_settings.access_log_location is defined %}
+{% if nginx_config_main_template.http_settings.access_log_location is sameas false or nginx_config_main_template.http_settings.access_log_location == "off" %}
+    access_log off;
+{% else %}
 {% for access_log in nginx_config_main_template.http_settings.access_log_location %}
     access_log  {{ access_log.location }}  {{ access_log.name }};
 {% endfor %}
+{% endif %}
+{% endif %}
 
     sendfile        on;