[Freeipa-devel] [PATCH 0052] Only specify the ipatokenuniqueid default in the add operation
Nathaniel McCallum
npmccallum at redhat.com
Tue May 13 16:55:19 UTC 2014
On Tue, 2014-05-13 at 16:47 +0200, Jan Cholasta wrote:
> On 12.5.2014 20:50, Nathaniel McCallum wrote:
> > On Mon, 2014-05-12 at 18:40 +0200, Misnyovszki Adam wrote:
> >> On Tue, 06 May 2014 11:46:14 -0400
> >> Nathaniel McCallum <npmccallum at redhat.com> wrote:
> >>
> >>> On Tue, 2014-05-06 at 11:38 -0400, Nathaniel McCallum wrote:
> >>>> On Tue, 2014-05-06 at 17:34 +0200, Petr Vobornik wrote:
> >>>>> On 6.5.2014 17:13, Nathaniel McCallum wrote:
> >>>>>> On Tue, 2014-05-06 at 17:04 +0200, Petr Vobornik wrote:
> >>>>>>> On 6.5.2014 16:51, Nathaniel McCallum wrote:
> >>>>>>>> Specifying the default in the LDAP Object causes the
> >>>>>>>> parameter to be specified for non-add operations. This is
> >>>>>>>> especially problematic when performing the modify operation
> >>>>>>>> as it causes the primary key to change for every modification.
> >>>>>>>>
> >>>>>>>> https://fedorahosted.org/freeipa/ticket/4227
> >>>>>>>>
> >>>>>>>
> >>>>>>> shouldn't removal of `autofill=True,` be enough?
> >>>>>>
> >>>>>> Removing autofill=True results in the default not being used
> >>>>>> for the otptoken-add operation. That may be a different bug
> >>>>>> (I'm not sure what the expectation of autofill is).
> >>>>>>
> >>>>>> Nathaniel
> >>>>>>
> >>>>>
> >>>>> Seems to work form me with:
> >>>>>
> >>>>> diff --git a/ipalib/plugins/otptoken.py
> >>>>> b/ipalib/plugins/otptoken.py index f68ea7d..623f1f1 100644
> >>>>> --- a/ipalib/plugins/otptoken.py
> >>>>> +++ b/ipalib/plugins/otptoken.py
> >>>>> @@ -121,9 +121,7 @@ class otptoken(LDAPObject):
> >>>>> cli_name='id',
> >>>>> label=_('Unique ID'),
> >>>>> default_from=lambda: unicode(uuid.uuid4()),
> >>>>> - autofill=True,
> >>>>> primary_key=True,
> >>>>> - flags=('optional_create'),
> >>>>> ),
> >>>>> StrEnum('type?',
> >>>>> label=_('Type'),
> >>>>
> >>>> Doing this causes the ipa otptoken-add command to prompt for the
> >>>> Unique ID. This may be the desired behavior, but it is not how it
> >>>> worked previously (no prompt).
> >>>
> >>> Here is an alternate patch for this second approach. I have no strong
> >>> opinion on the correct behavior here.
> >>>
> >>> Nathaniel
> >>
> >> IMO you should update API.txt with ./makeapi
> >
> > Running ./makeapi results in no changes to API.txt.
>
> This is not right, there *are* changes in the API and build fails for me
> becase API.txt is not updated.
I think maybe I ran it from the wrong branch. Fixed.
Nathaniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-npmccallum-0052a.1-Only-use-the-ipatokenuniqueid-default-in-the-add-ope.patch
Type: text/x-patch
Size: 5273 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140513/ae980038/attachment.bin>
More information about the Freeipa-devel
mailing list