On Wed, Feb 02, 2011 at 01:50:40PM -0500, Stephen Gallagher wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 02/02/2011 11:22 AM, Stephen Gallagher wrote: > > On 02/02/2011 04:23 AM, Sumit Bose wrote: > >> On Tue, Feb 01, 2011 at 04:18:27PM -0500, Stephen Gallagher wrote: > >> Patch 0001: Make the domain argument mandatory in sss_obfuscate > >> It doesn't make sense to set a "default" domain. We should require > >> that the domain always be specified. > > > >>> ACK > > > > > >> Patch 0002: Gracefully handle permission errors in sss_obfuscate > >> Don't show a traceback if not run as root. > >> Replaces patch from "Fixing traceback call messages for sss_obfuscate > >> command" > > > >>> Just some nitpicking: > > > >>> - SSSDConfig.SSSDConfig() only throws an IOError if it cannot read the > >>> schema files. If there is a setup where this really happens then I > >>> think "Permission denied when reading current configuration" is > >>> misleading, because changing the permissions on sssd.conf wouldn't > >>> help. > >>> - Not related to this patch, but maybe you want to fix this too. If > >>> there is no config file name given on the command line > >>> options.filename is empty and if SSSDConfig.import_config cannot read > >>> the default config file the message "Cannot open config file None" is > >>> printed. > >>> - Even less related, but still important. -p is broken, it takes the > >>> codepath of -s so you have to press CTRL-D to leave sss_obfuscate but > >>> even then the value of -p is not taken, but the empty string is > >>> obfuscated. If would vote to remove this option, because it can easy > >>> be substituted by 'echo -n password | sss_obfuscate -d domain -s' > > > > All good suggestions. I didn't address -p, as Gowrishankar has submitted > > a patch to address this. > > > > I've updated my patches accordingly, and added a third patch that makes > > the SSSDConfig API internal configuration files world-readable. > > > > > New patches attached corrects a mistake where I was adding the execute > bit to sssd.conf (noticed by Sumit on IRC) >
ACK bye, Sumit > > - -- > Stephen Gallagher > RHCE 804006346421761 > > Delivering value year after year. > Red Hat ranks #1 in value among software vendors. > http://www.redhat.com/promo/vendor/ > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.11 (GNU/Linux) > Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ > > iEYEARECAAYFAk1Jp38ACgkQeiVVYja6o6PFKQCeO6rZw3/+wJZ1H3cOPsHWNUtY > TAEAnin0+ylYNhQEhIASZs2pbgR+k9Xt > =iE24 > -----END PGP SIGNATURE----- _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel