<div dir="ltr">Nope. Somehow in my head it felt cleaner. Updated patched attached.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 23, 2014 at 1:18 AM, Jan Cholasta <span dir="ltr"><<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 23.7.2014 01:01, Gabe Alford wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
Forgot about --trust-secret. Here is an updated patch.<br>
<br>
<br>
On Mon, Jul 21, 2014 at 2:31 AM, Jan Cholasta <<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a><br></div><div class="">
<mailto:<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a>>> wrote:<br>
<br>
    On 21.7.2014 10:28, Martin Kosek wrote:<br>
<br>
        On 07/21/2014 09:56 AM, Jan Cholasta wrote:<br>
<br>
            Hi,<br>
<br>
            On 16.7.2014 05:48, Gabe Alford wrote:<br>
<br>
                Hello,<br>
<br>
                Adds AD admin and password to interactive commands.<br></div>
                <a href="https://fedorahosted.org/__freeipa/ticket/3034" target="_blank">https://fedorahosted.org/__<u></u>freeipa/ticket/3034</a><div class=""><br>
                <<a href="https://fedorahosted.org/freeipa/ticket/3034" target="_blank">https://fedorahosted.org/<u></u>freeipa/ticket/3034</a>><br>
<br>
                Thanks,<br>
<br>
                Gabe<br>
<br>
<br>
            I think that instead of making the parameters mandatory, you<br>
            should instead set<br>
            alwaysask=True on them.<br>
<br>
            Honza<br>
<br>
<br>
        Trust can be established either with user+password options OR with<br>
        --trust-secret option - i.e. you cannot use mandatory options<br>
        nor alwaysask.<br>
<br>
<br>
    Ah, right.<br>
<br>
<br>
<br>
        This would rather lead to interactive_prompt_callback checking<br>
        if any of<br>
        authentication method is passed and asking for them if they aren't.<br>
<br>
<br>
    +1<br>
<br>
<br>
        Martin<br>
<br>
<br>
<br>
    --<br>
    Jan Cholasta<br>
<br>
<br>
</div></blockquote>
<br>
I don't think using an extra function to update a value in a dictionary is very beneficial, is there a reason not to use "kw[X] = self.prompt_param(self.params[<u></u>X])" directly?<span class="HOEnZb"><font color="#888888"><br>

<br>
-- <br>
Jan Cholasta<br>
</font></span></blockquote></div><br></div>