<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 16.08.2016 17:35, Tomas Krizek
wrote:<br>
</div>
<blockquote
cite="mid:e7efe1ef-fbfa-7c26-b02e-c2e9e62ed2a4@redhat.com"
type="cite">Hi,
<br>
<br>
the attached patch fixes an error message when user provides an
empty key while adding otp token.
<br>
<br>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/6200">https://fedorahosted.org/freeipa/ticket/6200</a>
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
I'm curious why we don't fix it here:<br>
<br>
OTPTokenKey('ipatokenotpkey?',<br>
cli_name='key',<br>
label=_('Key'),<br>
doc=_('Token secret (Base32; default: random)'),<br>
default_from=lambda: os.urandom(KEY_LENGTH),<br>
autofill=True,<br>
flags=('no_display', 'no_update', 'no_search'),<br>
),<br>
<br>
<br>
If OTPTokenKey is mandratory, it should be required param (autofill
should work in this case too)<br>
<br>
Martin^2<br>
</body>
</html>