jvonau suggested changes

roles/captiveportal/tasks/main.yml

@@ -38,8 +38,8 @@
 
 - name: Copy uWSGI config file
   template:
-    src: roles/captiveportal/templates/captiveportal.ini
-    dest: /opt/iiab/captiveportal/
+    src: roles/captiveportal/templates/captiveportal.ini.j2
+    dest: /opt/iiab/captiveportal/captiveportal.ini
 
 - name: Copy unit file for uWSGI service
   template:
@@ -75,6 +75,13 @@
     state: link
   when: captiveportal_enabled | bool
 
+- name: Disable nginx to location definitions for checkurls
+  file:
+    src: /etc/nginx/sites-available/capture.conf
+    path: /etc/nginx/sites-enabled/capture.conf
+    state: absent
+  when: not captiveportal_enabled | bool
+
 - name: Make sure dnsmasq is not diverting if not captiveportal_enabled
   file:
     path: /etc/dnsmasq.d/capture

roles/captiveportal/templates/captiveportal.ini

@@ -1,12 +0,0 @@
-[uwsgi]
-   uid = www-data
-   gid = www-data
-   http-socket = :9090
-   chdir = /opt/iiab/captiveportal
-   wsgi-file = capture-wsgi.py
-   #wsgi-file = very_simple.py
-   master = true
-   plugins = python3
-   log-to = /var/log/uwsgi/app/captiveportal.log
-   #die-on-term = true
-   py-autoreload = 2

roles/captiveportal/templates/captiveportal.ini.j2

@@ -0,0 +1,10 @@
+[uwsgi]
+   uid = {{ apache_user }}
+   gid = {{ apache_user }}
+   http-socket = {{ captiveportal_port }}
+   chdir = /opt/iiab/captiveportal
+   wsgi-file = capture-wsgi.py
+   master = true
+   plugins = python3
+   log-to = /var/log/apache2/captiveportal.log
+   py-autoreload = 2