[Freeipa-devel] [PATCH] 21 Fix directory manager password validation in ipa-nis-manage
Martin Kosek
mkosek at redhat.com
Mon Jun 13 11:02:04 UTC 2011
On Mon, 2011-06-13 at 12:32 +0200, Jan Cholasta wrote:
> On 9.6.2011 14:50, Martin Kosek wrote:
> > On Wed, 2011-06-08 at 15:21 +0200, Jan Cholasta wrote:
> >> https://fedorahosted.org/freeipa/ticket/1283
> >> https://fedorahosted.org/freeipa/ticket/1284
> >>
> >> Honza
> >
> > Patch works fine, but I'd like to improve code quality a bit. Please
> > don't call sys.exit() from get_dirman_password(). It doesn't make really
> > sense.
> >
> > I suggest just returning None in that case and then exiting in the main
> > function. Or raising a proper exception and then exiting in the main
> > function. The get_dirman_password() function can then be later reused
> > easily.
>
> Good point. Fixed.
ACK. Pushed to master, ipa-2-0.
Martin
More information about the Freeipa-devel
mailing list