On 24.08.2016 13:32, Tomas Krizek wrote:
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
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:
the attached patch fixes an error message when user provides an
empty key while adding otp token.
I'm curious why we don't fix it here:
doc=_('Token secret (Base32; default: random)'),
flags=('no_display', 'no_update', 'no_search'),
If OTPTokenKey is mandratory, it should be required param (autofill
should work in this case too)
You changed API, you must regenerate API.txt (./makeapi) and
minor version in VERSION file
Option 'ipatokenotpkey?' in command 'otptoken_add/1' in API file
Options count in otptoken_add of 22 doesn't match expected: 23
Option ipatokenotpkey of command otptoken_add in ipalib, not in
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)
Pushed to master: 6f9a029bf5d33e6c8267cb330bd48033c5517188
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code