Install RTMP only when using NGINX Plus (#91)

* Install RTMP only when using NGINX Plus

* Make GeoIP and NJS module tasks idempotent

tasks/modules/install-geoip.yml

@@ -12,10 +12,11 @@
   when: nginx_type == "plus"
 
 - name: "(Setup: All NGINX) Load NGINX GeoIP Module"
-  blockinfile:
+  lineinfile:
     path: /etc/nginx/nginx.conf
     insertbefore: BOF
-    block: |
+    line: "{{ item }}"
-      load_module modules/ngx_http_geoip_module.so;
+  with_items:
-      load_module modules/ngx_stream_geoip_module.so;
+    - load_module modules/ngx_http_geoip_module.so;
+    - load_module modules/ngx_stream_geoip_module.so;
   notify: "(Handler: All OSs) Reload NGINX"

tasks/modules/install-modules.yml

@@ -12,7 +12,7 @@
   when: nginx_modules.image_filter | default(false)
 
 - import_tasks: install-rtmp.yml
-  when: nginx_modules.rtmp | default(false)
+  when: nginx_modules.rtmp | default(false) and nginx_type == "plus"
 
 - import_tasks: install-xslt.yml
   when: nginx_modules.xslt | default(false)

tasks/modules/install-njs.yml

@@ -12,10 +12,11 @@
   when: nginx_type == "plus"
 
 - name: "(Setup: All NGINX) Load NGINX JavaScript Module"
-  blockinfile:
+  lineinfile:
     path: /etc/nginx/nginx.conf
     insertbefore: BOF
-    block: |
+    line: "{{ item }}"
-      load_module modules/ngx_http_js_module.so;
+  with_items:
-      load_module modules/ngx_stream_js_module.so;
+    - load_module modules/ngx_http_js_module.so;
+    - load_module modules/ngx_stream_js_module.so;
   notify: "(Handler: All OSs) Reload NGINX"

tasks/modules/install-rtmp.yml

@@ -1,15 +1,8 @@
 ---
-- name: "(Install: All OSs) Install NGINX Open Source RTMP Module"
-  package:
-    name: nginx-module-rtmp
-    state: present
-  when: nginx_type == "opensource"
-
 - name: "(Install: All OSs) Install NGINX Plus RTMP Module"
   package:
     name: nginx-plus-module-rtmp
     state: present
-  when: nginx_type == "plus"
 
 - name: "(Setup: All NGINX) Load NGINX RTMP Module"
   lineinfile: