diff options
author | Endi S. Dewata <edewata@redhat.com> | 2017-07-11 06:08:00 +0200 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2017-07-11 06:50:50 +0200 |
commit | de9337900e898da0a2de38c5d044722a26fcceee (patch) | |
tree | 386d7a730e3d91999b1e9fd0ed8b065cc0feff44 | |
parent | fe3e383ef4c45b35b8cc3ddeb3bfeb648ee322aa (diff) | |
download | pki-de9337900e898da0a2de38c5d044722a26fcceee.tar.gz pki-de9337900e898da0a2de38c5d044722a26fcceee.tar.xz pki-de9337900e898da0a2de38c5d044722a26fcceee.zip |
Fixed CertUtil.updateLocalRequest().
The CertUtil.updateLocalRequest() has been modified to return
silently if the certificate does not have a corresponding request
record.
https://pagure.io/dogtagpki/issue/2280
Change-Id: I0d225a6db84d2d719091dbd84ee76b73bfb4408c
-rw-r--r-- | base/server/cms/src/com/netscape/cms/servlet/csadmin/CertUtil.java | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/base/server/cms/src/com/netscape/cms/servlet/csadmin/CertUtil.java b/base/server/cms/src/com/netscape/cms/servlet/csadmin/CertUtil.java index c047aac24..a3e425ee5 100644 --- a/base/server/cms/src/com/netscape/cms/servlet/csadmin/CertUtil.java +++ b/base/server/cms/src/com/netscape/cms/servlet/csadmin/CertUtil.java @@ -355,11 +355,16 @@ public class CertUtil { CMS.debug("CertUtil.updateLocalRequest(" + certTag + ")"); + String reqId = config.getString("preop.cert." + certTag + ".reqId", null); + if (reqId == null) { + CMS.debug("CertUtil: cert has no request record"); + return; + } + ICertificateAuthority ca = (ICertificateAuthority) CMS.getSubsystem(ICertificateAuthority.ID); IRequestQueue queue = ca.getRequestQueue(); - RequestId rid = new RequestId(config.getString("preop.cert." + certTag + ".reqId")); - IRequest req = queue.findRequest(rid); + IRequest req = queue.findRequest(new RequestId(reqId)); if (!certReq.equals("")) { CMS.debug("CertUtil: updating cert request"); |