Skip to content

Commit

Permalink
Issue 4736 - lib389 - fix regression in certutil error checking
Browse files Browse the repository at this point in the history
Description: A regression in the previous commit accidentally called
certutil twice which triggered the CLI to prompt for the NSS database
password.  This broke CI tests, etc.

relates: #4736

Reviewed by: mreynolds (one line commit rule)
  • Loading branch information
mreynolds389 committed Aug 10, 2021
1 parent b99236a commit 07b44fe
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/lib389/lib389/nss_ssl.py
Expand Up @@ -195,7 +195,6 @@ def reinit(self):
cmd = ['/usr/bin/certutil', '-N', '-d', self._certdb, '-f', '%s/%s' % (self._certdb, PWD_TXT)]
self._generate_noise('%s/noise.txt' % self._certdb)
self.log.debug("nss cmd: %s", format_cmd_list(cmd))
result = ensure_str(check_output(cmd, stderr=subprocess.STDOUT))
try:
result = ensure_str(check_output(cmd, stderr=subprocess.STDOUT))
except subprocess.CalledProcessError as e:
Expand Down

0 comments on commit 07b44fe

Please sign in to comment.