<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 23.06.2016 08:52, Stanislav Laznicka
wrote:<br>
</div>
<blockquote
cite="mid:541c5c87-6121-9ff5-1081-306fac42a123@redhat.com"
type="cite">On 06/23/2016 08:09 AM, Jan Cholasta wrote:
<br>
<blockquote type="cite">On 22.6.2016 16:22, Stanislav Laznicka
wrote:
<br>
<blockquote type="cite">Hello,
<br>
<br>
Please see the patch attached that fixes the issue from
<br>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/5965">https://fedorahosted.org/freeipa/ticket/5965</a>. The patch took
me quite a
<br>
while to create as I thought something was wrong with the
SshExec class
<br>
which actually was where the password was required.
<br>
</blockquote>
<br>
"The nss_db variable didn't go through the proper
initialization"
<br>
<br>
You are going to have to be more specific, because the variable
is properly initialized right here:
<br>
<br>
with certdb.NSSDatabase(nss_dir) as nss_db:
<br>
<br>
And the nss_db.secdir attribute used in the api.bootstrap() call
is properly initialized in NSSDatabase():
<br>
<br>
def __init__(self, nssdir=None):
<br>
if nssdir is None:
<br>
self.secdir = tempfile.mkdtemp()
<br>
self._is_temporary = True
<br>
else:
<br>
self.secdir = nssdir
<br>
self._is_temporary = False
<br>
<br>
</blockquote>
You're right, the commit message was rather generic. Hopefully
this new one will be better.
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
Works for me, if Honza agree this can be pushed.<br>
</body>
</html>