From 1c3e6844bc7ae8c8bdbab10b725bbb2292e57a2d Mon Sep 17 00:00:00 2001 From: Robert Fairburn Date: Wed, 9 Jul 2014 13:35:07 -0500 Subject: [PATCH] Fix broken logic surrounding includes and Debian defaults --- sudoers/files/sudoers | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/sudoers/files/sudoers b/sudoers/files/sudoers index 088772b..1126685 100644 --- a/sudoers/files/sudoers +++ b/sudoers/files/sudoers @@ -1,16 +1,16 @@ {%- if (not included) %} {%- set sudoers = pillar.get('sudoers', {}) %} {%- if grains['os_family'] == 'Debian' %} - {%- set defaults = sudoers.get('defaults', [ + {%- set defaults = sudoers.get('defaults', {'generic': [ 'env_reset', 'mail_badpass', 'secure_path="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"' - ]) %} + ]}) %} {%- set users = sudoers.get('users', {'root': 'ALL=(ALL:ALL) ALL'}) %} {%- set groups = sudoers.get('groups', {'sudo': 'ALL=(ALL:ALL) ALL'}) %} {%- else %} {%- set defaults = sudoers.get('defaults', {}) %} - {%- set generic_defaults = defaults.get('generic', {}) %} + {%- set generic_defaults = defaults.get('generic', []) %} {%- set user_list_defaults = defaults.get('user_list', {}) %} {%- set host_list_defaults = defaults.get('host_list', {}) %} {%- set command_list_defaults = defaults.get('command_list', {}) %} @@ -21,6 +21,11 @@ {%- set includedir = sudoers.get('includedir', '/etc/sudoers.d') -%} {%- else %} {%- set defaults = sudoers.get('defaults', []) %} + {%- set generic_defaults = defaults.get('generic', []) %} + {%- set user_list_defaults = defaults.get('user_list', {}) %} + {%- set host_list_defaults = defaults.get('host_list', {}) %} + {%- set command_list_defaults = defaults.get('command_list', {}) %} + {%- set runas_list_defaults = defaults.get('runas_list', {}) %} {%- set users = sudoers.get('users', {}) %} {%- set groups = sudoers.get('groups', {}) %} {%- set includedir = sudoers.get('includedir', None) %}