From ea76d0d84f17e8bebaa69dd9ec4fd04287548616 Mon Sep 17 00:00:00 2001 From: Scott Reeves Date: Fri, 24 Oct 2014 22:57:44 -0400 Subject: [PATCH] Remove trailing slash from sudoers_dir --- users/init.sls | 18 +++++++++--------- users/map.jinja | 8 ++++---- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/users/init.sls b/users/init.sls index 92f940d..abf84bf 100644 --- a/users/init.sls +++ b/users/init.sls @@ -148,7 +148,7 @@ include: sudoer-{{ name }}: file.managed: - - name: {{ users.sudoers_dir }}{{ name }} + - name: {{ users.sudoers_dir }}/{{ name }} - user: root - group: {{ users.root_group }} - mode: '0440' @@ -163,10 +163,10 @@ sudoer-{{ name }}: # Specify the rule via an env var to avoid shell quoting issues. - rule: "{{ name }} {{ rule }}" - require_in: - - file: {{ users.sudoers_dir }}{{ name }} + - file: {{ users.sudoers_dir }}/{{ name }} {% endfor %} -{{ users.sudoers_dir }}{{ name }}: +{{ users.sudoers_dir }}/{{ name }}: file.managed: - contents: | {%- for rule in user['sudo_rules'] %} @@ -177,9 +177,9 @@ sudoer-{{ name }}: - file: sudoer-{{ name }} {% endif %} {% else %} -{{ users.sudoers_dir }}{{ name }}: +{{ users.sudoers_dir }}/{{ name }}: file.absent: - - name: {{ users.sudoers_dir }}{{ name }} + - name: {{ users.sudoers_dir }}/{{ name }} {% endif %} {% endfor %} @@ -197,17 +197,17 @@ sudoer-{{ name }}: {% else %} user.absent {% endif -%} -{{ users.sudoers_dir }}{{ name }}: +{{ users.sudoers_dir }}/{{ name }}: file.absent: - - name: {{ users.sudoers_dir }}{{ name }} + - name: {{ users.sudoers_dir }}/{{ name }} {% endfor %} {% for user in pillar.get('absent_users', []) %} {{ user }}: user.absent -{{ users.sudoers_dir }}{{ user }}: +{{ users.sudoers_dir }}/{{ user }}: file.absent: - - name: {{ users.sudoers_dir }}{{ user }} + - name: {{ users.sudoers_dir }}/{{ user }} {% endfor %} {% for group in pillar.get('absent_groups', []) %} diff --git a/users/map.jinja b/users/map.jinja index 987e22e..0779fd4 100644 --- a/users/map.jinja +++ b/users/map.jinja @@ -1,7 +1,7 @@ # vim: sts=2 ts=2 sw=2 et ai {% set users = salt['grains.filter_by']({ 'Debian': { - 'sudoers_dir': '/etc/sudoers.d/', + 'sudoers_dir': '/etc/sudoers.d', 'sudoers_file': '/etc/sudoers', 'root_group': 'root', 'shell': '/bin/bash', @@ -10,7 +10,7 @@ 'sudo_package': 'sudo', }, 'Gentoo': { - 'sudoers_dir': '/etc/sudoers.d/', + 'sudoers_dir': '/etc/sudoers.d', 'sudoers_file': '/etc/sudoers', 'root_group': 'root', 'shell': '/bin/bash', @@ -19,7 +19,7 @@ 'sudo_package': 'app-admin/sudo', }, 'FreeBSD': { - 'sudoers_dir': '/usr/local/etc/sudoers.d/', + 'sudoers_dir': '/usr/local/etc/sudoers.d', 'sudoers_file': '/usr/local/etc/sudoers', 'root_group': 'wheel', 'shell': '/bin/csh', @@ -28,7 +28,7 @@ 'sudo_package': 'sudo', }, 'default': { - 'sudoers_dir': '/etc/sudoers.d/', + 'sudoers_dir': '/etc/sudoers.d', 'sudoers_file': '/etc/sudoers', 'root_group': 'root', 'shell': '/bin/bash',