[SSSD] [PATCH] Add vetoed_shells to sssd.api.conf

Jakub Hrozek jhrozek at redhat.com
Mon Sep 19 19:36:40 UTC 2011


On Fri, Sep 16, 2011 at 09:51:35AM -0400, Stephen Gallagher wrote:
> On Fri, 2011-09-16 at 10:17 +0300, Marko Myllynen wrote:
> > Hi,
> > 
> > while testing vetoed_shells I happened to notice it is missing from
> > sssd.api.conf, the attached patch fixes this.
> > 
> > However, there seems to be also ~dozen other configuration options
> > missing, not sure should those all be added, too.
> > 
> > The list of missing configuration options after this patch is:
> > 
> > localhost:~/sssd.git> for t in `grep -Eo '<term>[a-z_]*'
> > src/man/sssd.conf.5.xml | sed -e 's,<term>,,g'`; do grep -q $t
> > src/config/etc/sssd.api.conf || echo $t ; done
> > config_file_version
> 
> * This is a special option and is hard-coded into the API itself and
> doesn't need to be specified in the config files.
> 
> > try_inotify
> Yeah, this needs to be added.
> 
> > entry_cache_nowait_percentage
> This was there, but with a typo.
> 
> > proxy_pam_target
> > proxy_lib_name
> These two exist in the sssd.api.d/sssd-proxy.conf
> 
> > default_shell
> > base_directory
> 
> These two exist in the sssd.api.d/sssd-local.conf
> 
> > create_homedir
> > remove_homedir
> > homedir_umask
> > skel_dir
> > mail_dir
> > userdel_cmd
> > 
> 
> These are all options that belong in sssd.api.d/sssd-local.conf but
> aren't there currently.
> 
> 
> New patch attached should fix all of these. I amended Marko's patch (but
> left him the attribution)
> 

Ack



More information about the sssd-devel mailing list