formatting

roles/caddy/tasks/main.yml

@@ -6,7 +6,7 @@
       - debian-keyring
       - debian-archive-keyring
       - apt-transport-https
-    update_cache: yes
+    update_cache: true
 
 - name: Add caddy repo signing key
   apt_key:
@@ -30,7 +30,7 @@
     marker_end: '# https://caddyserver.com/docs/caddyfile'
     state: absent
     validate: 'caddy validate --config %s --adapter caddyfile'
-    backup: yes
+    backup: true
 
 - name: Put caddy general config in place
   blockinfile:
@@ -41,5 +41,5 @@
         email {{ admin_email }}
       }
     validate: 'caddy validate --config %s --adapter caddyfile'
-    backup: yes
+    backup: true
   notify: reload caddy

roles/locale/defaults/main.yml

@@ -7,4 +7,3 @@ gen_locales:
 lang: en_US.UTF-8
 lc_messages: "{{ lang }}"
 language: "en_US:en"
-

roles/locale/tasks/main.yml

@@ -17,4 +17,4 @@
   template:
     src: locale.j2
     dest: "{{ locale_config }}"
-
+    mode: 0644

roles/packages/tasks/apt.yml

@@ -4,10 +4,9 @@
   apt:
     name: "{{ install_packages }}"
     state: present
-    update_cache: yes
+    update_cache: true
 
 - block:
-
     - name: Enable backports
       apt_repository:
         repo: "deb http://deb.debian.org/debian {{ ansible_distribution_release }}-backports main"
@@ -18,6 +17,4 @@
         name: "{{ backports_packages }}"
         state: present
         default_release: "{{ ansible_distribution_release }}-backports"
-
   when: backports_packages is defined
-

roles/packages/tasks/main.yml

@@ -13,4 +13,3 @@
     name: "{{ delete_packages }}"
     state: absent
   when: delete_packages is defined
-

roles/packages/tasks/other.yml

@@ -4,4 +4,3 @@
   package:
     name: "{{ install_packages }}"
     state: present
-

roles/reverse_proxy/tasks/main.yml

@@ -5,12 +5,14 @@
     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
+    - name: Split legacy proxy_target to protocol and target
       set_fact:
         proxy_target_split_protocol: "{{ proxy_target.split('://') }}"
-  - set_fact:
+    - name: Split target further to host and port
+        set_fact:
         proxy_target_split_host: "{{ (proxy_target_split_protocol | last).split(':') }}"
-  - set_fact:
+    - name: Set host and port variables
+      set_fact:
         proxy_target_host: "{{ proxy_target_split_host[0] }}"
         proxy_target_port: "{{ proxy_target_split_host[1] }}"
 
@@ -18,7 +20,6 @@
       set_fact:
         proxy_target_protocol: "{{ proxy_target_split_protocol[0] }}"
       when: proxy_target_split_protocol | length == 2
-
   when: proxy_target is defined and proxy_target_port is not defined
 
 - name: configure vhost for reverse proxy
@@ -26,4 +27,3 @@
     name: vhost
   vars:
     vhost_type: reverse_proxy
-

roles/reverse_proxy/vars/main.yml

@@ -2,4 +2,3 @@
 
 reverse_proxy_type: caddy
 web_server: "{{ reverse_proxy_type }}"
-

roles/ssh/tasks/main.yml

@@ -12,4 +12,3 @@
     PermitRootLogin: 'prohibit-password'
     PasswordAuthentication: "{{ 'yes' if sshd_password_auth else 'no' }}"
     X11Forwarding: "{{ 'yes' if sshd_x11_forwarding else 'no' }}"
-

roles/vhost/meta/argument_specs.yml

@@ -49,4 +49,3 @@ argument_specs:
         choices:
           - temporary
           - permanent
-

roles/vhost/meta/main.yml

@@ -3,4 +3,3 @@
 dependencies:
   - role: caddy
     when: web_server == 'caddy'
-

roles/vhost/tasks/caddy.yml

@@ -22,4 +22,3 @@
     validate: 'caddy validate --config %s --adapter caddyfile'
     backup: true
   notify: reload caddy
-