From f25cec613a4cf7aecccf392ad0bb81570a5ea527 Mon Sep 17 00:00:00 2001
From: Shawn Butts <shawnbutts@users.noreply.github.com>
Date: Mon, 28 Oct 2013 16:39:55 -0400
Subject: [PATCH] better sudoers support & default gid

add support for sudouser being False.
change to adding sudoers config to /etc/sudoers.d/<user>
adding the removal of /etc/sudoers.d/<user> on user removal or switching to sudouser being removed or set to false
---
 users/init.sls | 30 ++++++++++++++++++++++--------
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/users/init.sls b/users/init.sls
index 6a16d36..b264c99 100644
--- a/users/init.sls
+++ b/users/init.sls
@@ -25,6 +25,7 @@ include:
       - group: {{ name }}
   group.present:
     - name: {{ name }}
+    - gid: {{ user['uid'] }}
   user.present:
     - name: {{ name }}
     - home: {{ home }}
@@ -101,15 +102,25 @@ ssh_auth_{{ name }}_{{ loop.index0 }}:
 {% endfor %}
 {% endif %}
 
-{% if 'sudouser' in user %}
-sudoer-{{ name }}:
-    file.append:
-        - name: /etc/sudoers
-        - text:
-          - "{{ name }}    ALL=(ALL)  NOPASSWD: ALL"
-        - require:
-          - file: sudoer-defaults
 
+{% if 'sudouser' in user and user['sudouser'] %}
+sudoer-{{ name }}:
+  file.managed:
+    - name: /etc/sudoers.d/{{ name }}
+    - user: root
+    - group: root
+    - mode: '0440'
+/etc/sudoers.d/{{ name }}:
+  file.append:
+  - text:
+    - "{{ name }}    ALL=(ALL)  NOPASSWD: ALL"
+  - require:
+    - file: sudoer-defaults
+    - file: sudoer-{{ name }}
+{% else %}
+/etc/sudoers.d/{{ name }}:
+  file.absent:
+    - name: /etc/sudoers.d/{{ name }}
 {% endif %}
 
 {% endfor %}
@@ -117,4 +128,7 @@ sudoer-{{ name }}:
 {% for user in pillar.get('absent_users', []) %}
 {{ user }}:
   user.absent
+/etc/sudoers.d/{{ user }}:
+  file.absent:
+    - name: /etc/sudoers.d/{{ user }}
 {% endfor %}