[Freeipa-devel] [PATCH] 0003 Validate key in otptoken-add
Tomas Krizek
tkrizek at redhat.com
Wed Aug 24 11:32:40 UTC 2016
Fixed the typo in error message.
On 08/23/2016 12:15 PM, Tomas Krizek wrote:
> In that case, the first version of the patch solves the issue.
>
> I'm attaching the patch once again, but it's the same as the one in
> the original message.
>
>
> On 08/23/2016 11:53 AM, Jan Cholasta wrote:
>> On 22.8.2016 19:08, Tomas Krizek wrote:
>>> 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')
>>
>> NACK, this is a backward incompatible change.
>>
>> AFAICT the option should remain optional, see the doc string:
>>
>> Token secret (Base32; default: random)
>> ^^^^^^^^^^^^^^^
>>
>
>
>
--
Tomas Krizek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160824/e00119bf/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-tkrizek-0003-4-Validate-key-in-otptoken-add.patch
Type: text/x-patch
Size: 1045 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160824/e00119bf/attachment.bin>
More information about the Freeipa-devel
mailing list