Refactor handlers (#321)

Author: alessfg

CHANGELOG.md

@@ -20,6 +20,7 @@ FEATURES:
 
 ENHANCEMENTS:
 
+*   Added handlers to check for NGINX syntax validity and fail if any errors are detected.
 *   Major backend refactoring to reduce the number of files and tasks.
 *   You can now specify an `nginx_repository` for NGINX Plus too.
 *   Moved "constant" variables to `vars/main.yml`.

handlers/main.yml

@@ -1,28 +1,29 @@
 ---
 - name: "(Handler) Check NGINX"
   command: "nginx -t"
-  changed_when: false
+  register: config
+  ignore_errors: yes
+  listen: "(Handler) Run NGINX"
 
-- name: "(Handler) Systemd Daemon-Reload"
+- name: "(Handler) Print NGINX error if syntax check fails"
+  fail:
+    msg: "{{ config.stderr }}"
+  when: config.rc != 0
+  listen: "(Handler) Run NGINX"
+
+- name: "(Handler) Systemd daemon-reload"
   systemd:
     daemon_reload: yes
-  notify: "(Handler) Start NGINX"
-
-- name: "(Handler) Run NGINX"
-  block:
-    - name: "(Handler) Start NGINX"
-      service:
-        name: nginx
-        state: started
-        enabled: yes
-      notify: "(Handler) Check NGINX"
 
-    - name: "(Handler) Reload NGINX"
-      command: "nginx -s reload"
-      changed_when: false
+- name: "(Handler) Start/Reload NGINX"
+  service:
+    name: nginx
+    state: reloaded
+    enabled: yes
   when:
     - nginx_start | bool
     - not ansible_check_mode | bool
+  listen: "(Handler) Run NGINX"
 
 - name: "(Handler) Start NGINX Amplify agent"
   service:

tasks/config/modify-systemd.yml

@@ -20,7 +20,7 @@
   when:
     - not nginx_service_custom | bool
     - not nginx_service_clean | bool
-  notify: "(Handler) Systemd Daemon-Reload"
+  notify: "(Handler) Systemd daemon-reload"
 
 - name: "Customize override for NGINX systemd service"
   copy:
@@ -32,14 +32,14 @@
   when:
     - nginx_service_custom | bool
     - not nginx_service_clean | bool
-  notify: "(Handler) Systemd Daemon-Reload"
+  notify: "(Handler) Systemd daemon-reload"
 
 - name: "Remove override for NGINX systemd service"
   file:
     path: "{{ nginx_service_overridepath }}"
     state: absent
   when: nginx_service_clean | bool
-  notify: "(Handler) Systemd Daemon-Reload"
+  notify: "(Handler) Systemd daemon-reload"
 
 - name: "Modify systemd"
   debug:

tasks/config/template-config.yml

@@ -35,7 +35,7 @@
     backup: yes
     mode: 0644
   when: nginx_main_template_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure NGINX HTTP Directory Exists"
   file:
@@ -65,7 +65,7 @@
     mode: 0644
   with_dict: "{{ nginx_http_template }}"
   when: nginx_http_template_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Dynamically Generate NGINX Stub Status Configuration File"
   template:
@@ -74,7 +74,7 @@
     backup: yes
     mode: 0644
   when: nginx_status_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Dynamically Generate NGINX API Configuration File"
   template:
@@ -83,7 +83,7 @@
     backup: yes
     mode: 0644
   when: nginx_rest_api_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure NGINX Stream Directory Exists"
   file:
@@ -101,4 +101,4 @@
     mode: 0644
   with_dict: "{{ nginx_stream_template }}"
   when: nginx_stream_template_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"

tasks/config/upload-config.yml

@@ -19,7 +19,7 @@
     mode: 0644
   with_fileglob: "{{ nginx_html_upload_src }}"
   when: nginx_html_upload_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure NGINX Main Directory Exists"
   file:
@@ -35,7 +35,7 @@
     backup: yes
     mode: 0644
   when: nginx_main_upload_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure NGINX HTTP Directory Exists"
   file:
@@ -52,7 +52,7 @@
     mode: 0644
   with_fileglob: "{{ nginx_http_upload_src }}"
   when: nginx_http_upload_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure NGINX Stream Directory Exists"
   file:
@@ -69,7 +69,7 @@
     mode: 0644
   with_fileglob: "{{ nginx_stream_upload_src }}"
   when: nginx_stream_upload_enable | bool
-  notify: "(Handler) Reload NGINX"
+  notify: "(Handler) Run NGINX"
 
 - name: "(DEPRECATED) Ensure SSL Certificate Directory Exists"
   file:

tasks/opensource/install-alpine.yml

@@ -11,4 +11,4 @@
     repository: "{{ repository }}"
     state: "{{ nginx_state }}"
     update_cache: yes
-  notify: "(Handler) Start NGINX"
+  notify: "(Handler) Run NGINX"

tasks/opensource/install-bsd.yml

@@ -21,15 +21,15 @@
         name: "www/nginx{{ nginx_version | default('') }}"
         state: "{{ nginx_state }}"
       when: nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
 
     - name: "(FreeBSD) Install NGINX port"
       portinstall:
         name: "www/nginx{{ nginx_version | default('') }}"
         use_packages: "{{ nginx_bsd_portinstall_use_packages | default(omit) }}"
         state: "{{ nginx_state }}"
       when: not nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
   when: ansible_facts['system'] == "FreeBSD"
 
 - name: "(OpenBSD) Install NGINX"
@@ -40,23 +40,23 @@
         build: no
         state: "{{ nginx_state }}"
       when: nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
 
     - name: "(OpenBSD) Install NGINX port"
       openbsd_pkg:
         name: "nginx{{ nginx_version | default('') }}"
         build: yes
         state: "{{ nginx_state }}"
       when: not nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
   when: ansible_facts['system'] == "OpenBSD"
 
 - name: "(NetBSD) Install NGINX"
   block:
     - name: "NetBSD) Install NGINX package"
       command: "pkg_add www/nginx{{ nginx_version | default('') }}"
       when: nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
 
     - name: "(NetBSD) Install NGINX port"
       fail:
@@ -69,7 +69,7 @@
     - name: "Install NGINX package"
       command: "pkg install www/nginx{{ nginx_version | default('') }}"
       when: nginx_bsd_install_packages | bool
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
 
     - name: "Install NGINX port"
       fail:

tasks/opensource/install-debian.yml

@@ -11,4 +11,4 @@
   apt:
     name: "nginx{{ nginx_version | default('') }}"
     state: "{{ nginx_state }}"
-  notify: "(Handler) Start NGINX"
+  notify: "(Handler) Run NGINX"

tasks/opensource/install-oss.yml

@@ -25,7 +25,7 @@
         name: "nginx{{ nginx_version | default('') }}"
         state: "{{ nginx_state }}"
       when: nginx_install_from == "os_repository"
-      notify: "(Handler) Start NGINX"
+      notify: "(Handler) Run NGINX"
   when: ansible_facts['system'] | lower is not search('bsd')
 
 - name: "Install NGINX in Unix systems"

tasks/opensource/install-redhat.yml

@@ -30,4 +30,4 @@
     disablerepo: "*"
     enablerepo: "nginx"
     update_cache: yes
-  notify: "(Handler) Start NGINX"
+  notify: "(Handler) Run NGINX"