fix proxy var logic

Author: sjpb

ansible/roles/proxy/tasks/main.yml

@@ -1,9 +1,10 @@
 - name: Validate http_proxy definition
   ansible.builtin.assert:
-    that: proxy_http_proxy != '' # this is default if squid not active
+    that: proxy_http_proxy != ''
     fail_msg: >-
       Variable proxy_http_proxy cannot be the empty string for hosts in the
-      proxy group. See environment/common/inventory/group_vars/all/proxy.yml.
+      proxy group. See environments/common/inventory/group_vars/all/proxy.yml
+      for convenience variables to set this.
 - name: Define configuration in /etc/environment
   tags: proxy
   lineinfile:

environments/common/inventory/group_vars/all/proxy.yml

@@ -20,7 +20,7 @@ proxy_remove: false
 
 # full http proxy string - override if the above don't provide enough control:
 proxy_http_proxy: >-
-  {% if groups['squid'] | length > 0 %}
+  {% if proxy_http_address != '' %}
   http://
   {%- if proxy_basic_password -%}
   {{ proxy_basic_user }}:{{ proxy_basic_password }}@