Systemd tweaks (#28)

Author: alessfg

CHANGELOG.md

@@ -5,7 +5,8 @@
 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.
+*   Add a `nginx_app_protect_service_modify` variable to revert a breaking change introduced in 0.3.0 where timeouts would not be set by default.
+*   Set NGINX handler to `state: restarted` to prevent some compatibility issues when NGINX App Protect is installed on an instance already running NGINX beforehand.
 
 ## 0.3.0 (September 21, 2020)
 

defaults/main.yml

@@ -46,7 +46,8 @@ nginx_app_protect_start: true
 
 # Increase NGINX service timeout to accommodate ruleset loading from default 90s.
 # Default is commented out.
-# nginx_app_protect_timeout: 180
+nginx_app_protect_service_modify: true
+nginx_app_protect_timeout: 180
 
 # Creates basic configuration files and enables NGINX App Protect on the target host
 nginx_app_protect_configure: false

tasks/install/install-app-protect.yml

@@ -54,4 +54,4 @@
         group: root
         mode: 0644
       notify: (Handler - NGINX App Protect) Systemd daemon-reload
-  when: nginx_app_protect_timeout is defined
+  when: nginx_app_protect_service_modify | bool