Deprecate reverse_proxy role as all functionality is available in vhost

roles/reverse_proxy/README.md

@@ -1 +1 @@
-Sets up a reverse proxy defined by the `reverse_proxy_type` variable
+Deprecated

roles/reverse_proxy/defaults/main.yml

@@ -1,5 +0,0 @@
----
-
-reverse_proxy_type: caddy
-proxy_target_protocol: http
-proxy_target_host: localhost

roles/reverse_proxy/meta/main.yml

@@ -1,5 +1,2 @@
 ---
 
-dependencies:
-  - role: caddy
-    when: reverse_proxy_type == 'caddy'

roles/reverse_proxy/tasks/caddy.yml

@@ -1,20 +0,0 @@
----
-
-- name: Add caddy reverse proxy config
-  blockinfile:
-    path: /etc/caddy/Caddyfile
-    marker: "# {mark} ANSIBLE MANAGED BLOCK {{ vhost_id }}"
-    block: |
-      {{ vhost_domains | join(' ') }} {
-        reverse_proxy {{ proxy_target_protocol }}://{{ proxy_target_host }}:{{ proxy_target_port }} {
-        {% if proxy_target_protocol == 'https' and proxy_target_host == 'localhost' %}
-          transport http {
-            tls_insecure_skip_verify
-          }
-        {% endif %}
-        }
-      }
-    validate: 'caddy validate --config %s --adapter caddyfile'
-    backup: true
-  notify: reload caddy
-

roles/reverse_proxy/tasks/main.yml

@@ -1,5 +1,9 @@
 ---
 
+- name: Deprecation warning
+  debug:
+    msg: "uumas.general.reverse_proxy is deprecated. You should switch to uumas.general.vhost with vhost_type: reverse_proxy"
+
 - block:
   - name: Split legacy proxy_target to protocol, host and port
     set_fact:
@@ -17,6 +21,9 @@
 
   when: proxy_target is defined and proxy_target_port is not defined
 
-- name: Setup {{ vhost_id }} reverse proxy
+- name: configure vhost for reverse proxy
-  include_tasks: "{{ reverse_proxy_type }}.yml"
+  include_role: 
-  when: reverse_proxy_type != 'none'
+    name: vhost
+  vars:
+    vhost_type: reverse_proxy
+

roles/reverse_proxy/vars/main.yml

@@ -0,0 +1,5 @@
+---
+
+reverse_proxy_type: caddy
+web_server: "{{ reverse_proxy_type }}"
+