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

Stephen Gallagher sgallagh at redhat.com
Tue Sep 20 15:05:30 UTC 2011


On Mon, 2011-09-19 at 21:36 +0200, Jakub Hrozek wrote:
> 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

Pushed to master, sssd-1-6 and sssd-1-5.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110920/a0eef6aa/attachment.sig>


More information about the sssd-devel mailing list