Tweak handler to have a default state of restarted (#29)

Author: alessfg

CHANGELOG.md

@@ -5,6 +5,7 @@
 BUG FIXES:
 
 *   Rename handlers to use more specific role related naming and prevent namespace collision issues.
+*   Set NGINX handler to `state: restasted` to prevent some compatibility issues when NGINX App Protect is installed on an instance already running NGINX beforehand.
 
 ## 0.3.0 (September 21, 2020)
 

handlers/main.yml

@@ -3,10 +3,21 @@
   systemd:
     daemon_reload: true
 
+- name: (Handler - NGINX App Protect) Restart NGINX
+  service:
+    name: nginx
+    state: restarted
+    enabled: true
+  when:
+    - nginx_app_protect_start | bool
+    - not ansible_check_mode | bool
+  listen: (Handler - NGINX App Protect) Run NGINX
+
 - name: (Handler - NGINX App Protect) Check NGINX
   command: nginx -t
   register: config
   ignore_errors: true
+  changed_when: false
   listen: (Handler - NGINX App Protect) Run NGINX
 
 - name: (Handler - NGINX App Protect) Print NGINX error if syntax check fails
@@ -15,13 +26,3 @@
   failed_when: config.rc != 0
   when: config.rc != 0
   listen: (Handler - NGINX App Protect) Run NGINX
-
-- name: (Handler - NGINX App Protect) Start/reload NGINX
-  service:
-    name: nginx
-    state: reloaded
-    enabled: true
-  when:
-    - nginx_app_protect_start | bool
-    - not ansible_check_mode | bool
-  listen: (Handler - NGINX App Protect) Run NGINX