Fixed error handling in DoUnrevoke servlet.

The DoUnrevoke servlet has been modified to re-throw the EBaseException
such that the error message can be returned properly to the client.

Ticket #739

1 files changed, 3 insertions, 2 deletions

diff --git a/base/server/cms/src/com/netscape/cms/servlet/cert/DoUnrevoke.java b/base/server/cms/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
index 142fea3c8..723651eb6 100644
--- a/base/server/cms/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
+++ b/base/server/cms/src/com/netscape/cms/servlet/cert/DoUnrevoke.java
@@ -40,7 +40,6 @@ import com.netscape.certsrv.authorization.AuthzToken;
 import com.netscape.certsrv.authorization.EAuthzAccessDenied;
 import com.netscape.certsrv.base.EBaseException;
 import com.netscape.certsrv.base.IArgBlock;
-import com.netscape.certsrv.base.PKIException;
 import com.netscape.certsrv.ca.ICRLIssuingPoint;
 import com.netscape.certsrv.ca.ICertificateAuthority;
 import com.netscape.certsrv.common.ICMSRequest;
@@ -275,7 +274,7 @@ public class DoUnrevoke extends CMSServlet {
             processor.log(ILogger.LL_FAILURE, "Error " + e);
             processor.auditChangeRequest(ILogger.FAILURE);
 
-            throw new PKIException(e.getMessage());
+            throw e;
         }
 
         // change audit processing from "REQUEST" to "REQUEST_PROCESSED"
@@ -420,6 +419,8 @@ public class DoUnrevoke extends CMSServlet {
         } catch (EBaseException e) {
             processor.log(ILogger.LL_FAILURE, "Error " + e);
             processor.auditChangeRequestProcessed(ILogger.FAILURE);
+
+            throw e;
         }
     }