<div dir="ltr"><div><div><div><div><div>Patch 21:<br></div>Update per recommendation<br><br></div>Patch 22:<br></div>Added version option as well as updated the manpage.<br><br></div>Thanks,<br><br></div>Gabe<br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Jun 6, 2014 at 5:47 AM, Petr Viktorin <span dir="ltr"><<a href="mailto:pviktori@redhat.com" target="_blank">pviktori@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 06/06/2014 08:36 AM, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
Patch 21:<br>
This is a patch for <a href="https://fedorahosted.org/freeipa/ticket/3724" target="_blank">https://fedorahosted.org/<u></u>freeipa/ticket/3724</a><br>
</blockquote>
<br></div>
Nope, doesn't work for me: nolog_replace expects a string.<br>
<br>
 File "/usr/lib/python2.7/site-<u></u>packages/ipaserver/install/<u></u>installutils.py", line 639, in run_script<br>
    return_value = main_function()<br>
  File "/usr/sbin/ipa-upgradeconfig", line 1140, in main<br>
    certificate_renewal_update(ca)<u></u>,<br>
  File "/usr/sbin/ipa-upgradeconfig", line 691, in certificate_renewal_update<br>
    ca.configure_renewal()<br>
  File "/usr/lib/python2.7/site-<u></u>packages/ipaserver/install/<u></u>cainstance.py", line 1444, in configure_renewal<br>
    post_command='renew_ca_cert "%s"' % nickname)<br>
  File "/usr/lib/python2.7/site-<u></u>packages/ipapython/certmonger.<u></u>py", line 431, in dogtag_start_tracking<br>
    (stdout, stderr, returncode) = ipautil.run(args, nolog=[pin])<br>
  File "/usr/lib/python2.7/site-<u></u>packages/ipapython/ipautil.py"<u></u>, line 305, in run<br>
    arg_string = nolog_replace(args, nolog)<br>
  File "/usr/lib/python2.7/site-<u></u>packages/ipapython/ipautil.py"<u></u>, line 360, in nolog_replace<br>
    string = string.replace(nolog_value, 'XXXXXXXX')<br>
<br>
<br>
I'd recommend something like ' '.join(shell_quote(a) for a in args).<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Patch 22:<br>
This is part of <a href="https://fedorahosted.org/freeipa/ticket/4316" target="_blank">https://fedorahosted.org/<u></u>freeipa/ticket/4316</a> and adds<br>
the API to 'ipa env'<br>
<br>
One question that I had for adding 'ipa --version' is if using the<br>
default optparse version argument would be sufficient/intended i.e.<br>
version=('Version: %s, API_Version: %s' % (version, api_version) or if a<br>
custom optparse argument is needed.<br>
</blockquote>
<br></div>
I think the default will be just fine. Can you update the patch to add the option?<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Petrł<br>
<br>
</font></span></blockquote></div><br></div>