[Freeipa-devel] [PATCH] 0003 Validate key in otptoken-add
Tomas Krizek
tkrizek at redhat.com
Mon Aug 22 17:08:40 UTC 2016
I've attached the updated patch. Hopefully I didn't forget anything else
this time.
On 08/22/2016 05:48 PM, Martin Basti wrote:
>
> On 22.08.2016 10:22, Tomas Krizek wrote:
>>
>> Seems like a good idea, I'm attaching the updated patch. Autofill
>> does work when the param is required.
>>
>>
>> On 08/19/2016 04:19 PM, Martin Basti wrote:
>>>
>>>
>>>
>>> On 16.08.2016 17:35, Tomas Krizek wrote:
>>>> Hi,
>>>>
>>>> the attached patch fixes an error message when user provides an
>>>> empty key while adding otp token.
>>>>
>>>> https://fedorahosted.org/freeipa/ticket/6200
>>>>
>>>>
>>>>
>>>
>>> I'm curious why we don't fix it here:
>>>
>>> OTPTokenKey('ipatokenotpkey?',
>>> cli_name='key',
>>> label=_('Key'),
>>> doc=_('Token secret (Base32; default: random)'),
>>> default_from=lambda: os.urandom(KEY_LENGTH),
>>> autofill=True,
>>> flags=('no_display', 'no_update', 'no_search'),
>>> ),
>>>
>>>
>>> If OTPTokenKey is mandratory, it should be required param (autofill
>>> should work in this case too)
>>>
>>> Martin^2
>>
>> --
>> Tomas Krizek
>
> You changed API, you must regenerate API.txt (./makeapi) and increment
> minor version in VERSION file
>
> Option 'ipatokenotpkey?' in command 'otptoken_add/1' in API file not found
> Options count in otptoken_add of 22 doesn't match expected: 23
> Option ipatokenotpkey of command otptoken_add in ipalib, not in API file:
> OTPTokenKey('ipatokenotpkey', autofill=True, cli_name='key')
>
> Martin^2
--
Tomas Krizek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160822/70b3fe91/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-tkrizek-0003-3-Validate-key-in-otptoken-add.patch
Type: text/x-patch
Size: 2500 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160822/70b3fe91/attachment.bin>
More information about the Freeipa-devel
mailing list