|
 |
f332ec |
From 166a4b291a573d2c9f346a1b1051a2e9b45ff375 Mon Sep 17 00:00:00 2001
|
|
|
f332ec |
From: "Endi S. Dewata" <edewata@redhat.com>
|
|
|
f332ec |
Date: Wed, 16 Oct 2013 09:41:12 -0400
|
|
|
f332ec |
Subject: [PATCH 1/6] Fixed error handling in DoUnrevoke servlet.
|
|
|
f332ec |
|
|
|
f332ec |
The DoUnrevoke servlet has been modified to re-throw the EBaseException
|
|
|
f332ec |
such that the error message can be returned properly to the client.
|
|
|
f332ec |
|
|
|
f332ec |
Ticket #739
|
|
|
f332ec |
---
|
|
|
f332ec |
base/common/src/com/netscape/cms/servlet/cert/DoUnrevoke.java | 5 +++--
|
|
|
f332ec |
1 file changed, 3 insertions(+), 2 deletions(-)
|
|
|
f332ec |
|
|
|
f332ec |
diff --git a/base/common/src/com/netscape/cms/servlet/cert/DoUnrevoke.java b/base/common/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
|
|
|
f332ec |
index cca8381..2b30720 100644
|
|
|
f332ec |
--- a/base/common/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
|
|
|
f332ec |
+++ b/base/common/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
|
|
|
f332ec |
@@ -40,7 +40,6 @@ import com.netscape.certsrv.authorization.AuthzToken;
|
|
|
f332ec |
import com.netscape.certsrv.authorization.EAuthzAccessDenied;
|
|
|
f332ec |
import com.netscape.certsrv.base.EBaseException;
|
|
|
f332ec |
import com.netscape.certsrv.base.IArgBlock;
|
|
|
f332ec |
-import com.netscape.certsrv.base.PKIException;
|
|
|
f332ec |
import com.netscape.certsrv.ca.ICRLIssuingPoint;
|
|
|
f332ec |
import com.netscape.certsrv.ca.ICertificateAuthority;
|
|
|
f332ec |
import com.netscape.certsrv.dbs.certdb.CertId;
|
|
|
f332ec |
@@ -274,7 +273,7 @@ public class DoUnrevoke extends CMSServlet {
|
|
|
f332ec |
processor.log(ILogger.LL_FAILURE, "Error " + e);
|
|
|
f332ec |
processor.auditChangeRequest(ILogger.FAILURE);
|
|
|
f332ec |
|
|
|
f332ec |
- throw new PKIException(e.getMessage());
|
|
|
f332ec |
+ throw e;
|
|
|
f332ec |
}
|
|
|
f332ec |
|
|
|
f332ec |
// change audit processing from "REQUEST" to "REQUEST_PROCESSED"
|
|
|
f332ec |
@@ -419,6 +418,8 @@ public class DoUnrevoke extends CMSServlet {
|
|
|
f332ec |
} catch (EBaseException e) {
|
|
|
f332ec |
processor.log(ILogger.LL_FAILURE, "Error " + e);
|
|
|
f332ec |
processor.auditChangeRequestProcessed(ILogger.FAILURE);
|
|
|
f332ec |
+
|
|
|
f332ec |
+ throw e;
|
|
|
f332ec |
}
|
|
|
f332ec |
}
|
|
|
f332ec |
|
|
|
f332ec |
--
|
|
|
f332ec |
1.8.3.1
|
|
|
f332ec |
|