Merge pull request #71 from mdschmitt/use_nologin_shell

fix(defaults): Use nologin instead of /bin/false

prometheus/config/users.sls

@@ -18,7 +18,7 @@ prometheus-config-users-install-{{ name }}-user-present:
     - groups:
       - {{ name }}
               {%- if grains.os != 'Windows' %}
-    - shell: /bin/false
+    - shell: {{ p.shell }}
                   {%- if grains.kernel|lower == 'linux' %}
     - createhome: false
                     {%- if grains.get('osfinger', '') in ['Debian-11'] %}

prometheus/defaults.yaml

@@ -310,6 +310,8 @@ prometheus:
     rootuser: root
     rootgroup: root
 
+  shell: /usr/sbin/nologin
+
   retry_option:
     # https://docs.saltstack.com/en/latest/ref/states/requisites.html#retrying-states
     attempts: 3

prometheus/osfamilymap.yaml

@@ -406,6 +406,7 @@ Windows:
 
 MacOS:
   div: '/'
+  shell: /sbin/nologin
   dir:
     default: /etc/defaults
   identity: