<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><tt>I've attached the updated patch. Hopefully I didn't forget
anything else this time.</tt><br>
</p>
<tt><br>
</tt>
<div class="moz-cite-prefix"><tt>On 08/22/2016 05:48 PM, Martin
Basti wrote:</tt><tt><br>
</tt></div>
<blockquote
cite="mid:6b7ecf18-952c-1c2e-5a35-744dc3fe434b@redhat.com"
type="cite"><tt> </tt>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<tt><br>
</tt><tt> </tt>
<div class="moz-cite-prefix"><tt>On 22.08.2016 10:22, Tomas Krizek
wrote:</tt><br>
</div>
<blockquote
cite="mid:e2bcdb98-4adf-abcd-b2a0-b9b1aadfe3d8@redhat.com"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<p><tt>Seems like a good idea, I'm attaching the updated patch.
Autofill does work when the param is required.<br>
</tt></p>
<tt><br>
</tt>
<div class="moz-cite-prefix"><tt>On 08/19/2016 04:19 PM, Martin
Basti wrote:</tt><tt><br>
</tt></div>
<blockquote
cite="mid:ba5f5348-0106-f45e-2c2b-5a048e4e1634@redhat.com"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<p><tt><br>
</tt> </p>
<tt><br>
</tt>
<div class="moz-cite-prefix"><tt>On 16.08.2016 17:35, Tomas
Krizek wrote:</tt><tt><br>
</tt> </div>
<blockquote
cite="mid:e7efe1ef-fbfa-7c26-b02e-c2e9e62ed2a4@redhat.com"
type="cite"><tt>Hi, </tt><tt><br>
</tt> <tt><br>
</tt><tt> the attached patch fixes an error message when
user provides an empty key while adding otp token. </tt><tt><br>
</tt> <tt><br>
</tt> <tt><a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="https://fedorahosted.org/freeipa/ticket/6200">https://fedorahosted.org/freeipa/ticket/6200</a></tt><tt>
</tt><tt><br>
</tt> <tt><br>
</tt> <tt><br>
</tt>
<fieldset class="mimeAttachmentHeader"></fieldset>
<tt><br>
</tt> </blockquote>
<tt><br>
</tt><tt> I'm curious why we don't fix it here:</tt><tt><br>
</tt> <tt><br>
</tt><tt> OTPTokenKey('ipatokenotpkey?',</tt><tt><br>
</tt><tt> cli_name='key',</tt><tt><br>
</tt><tt> label=_('Key'),</tt><tt><br>
</tt><tt> doc=_('Token secret (Base32; default:
random)'),</tt><tt><br>
</tt><tt> default_from=lambda:
os.urandom(KEY_LENGTH),</tt><tt><br>
</tt><tt> autofill=True,</tt><tt><br>
</tt><tt> flags=('no_display', 'no_update',
'no_search'),</tt><tt><br>
</tt><tt> ),</tt><tt><br>
</tt> <tt><br>
</tt> <tt><br>
</tt><tt> If OTPTokenKey is mandratory, it should be required
param (autofill should work in this case too)</tt><tt><br>
</tt> <tt><br>
</tt><tt> Martin^2</tt><tt><br>
</tt> </blockquote>
<tt><br>
</tt>
<pre class="moz-signature" cols="72">--
Tomas Krizek</pre>
<tt> </tt></blockquote>
<tt> </tt><tt><br>
</tt><tt> You changed API, you must regenerate API.txt (./makeapi)
and increment minor version in VERSION file</tt><tt><br>
</tt><tt> </tt><tt><br>
</tt><tt> Option 'ipatokenotpkey?' in command 'otptoken_add/1' in
API file not found</tt><tt><br>
</tt><tt> Options count in otptoken_add of 22 doesn't match
expected: 23</tt><tt><br>
</tt><tt> Option ipatokenotpkey of command otptoken_add in ipalib,
not in API file:</tt><tt><br>
</tt><tt> OTPTokenKey('ipatokenotpkey', autofill=True,
cli_name='key')</tt><tt><br>
</tt><tt> </tt><tt><br>
</tt><tt> Martin^2</tt><br>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Tomas Krizek</pre>
</body>
</html>