chg: Avoid dirtying the environment with Thread Interruption exceptions if nobody...

[gigi.git] / src / org / cacert / gigi / api / RevokeCertificate.java

diff --git a/src/org/cacert/gigi/api/RevokeCertificate.java b/src/org/cacert/gigi/api/RevokeCertificate.java
index 404d73ce82afbcb1da3ecb7e7babeb5c9eb57109..2f15511afe69af95179e9678b6a6f231b15918b9 100644 (file)
--- a/src/org/cacert/gigi/api/RevokeCertificate.java
+++ b/src/org/cacert/gigi/api/RevokeCertificate.java
@@ -21,31 +21,32 @@ public class RevokeCertificate extends APIPoint {
             resp.sendError(500, "Error, POST required.");
             return;
         }
+
         if (req.getQueryString() != null) {
             resp.sendError(500, "Error, no query String allowed.");
             return;
         }
+
         String tserial = req.getParameter("serial");
         if (tserial == null) {
             resp.sendError(500, "Error, no Serial found");
             return;
         }
-        try {
-            Certificate c = Certificate.getBySerial(tserial);
-            if (c == null || c.getOwner() != u) {
-                resp.sendError(403, "Access Denied");
-                return;
-            }
-            Job job = c.revoke();
-            job.waitFor(60000);
-            if (c.getStatus() != CertificateStatus.REVOKED) {
-                resp.sendError(510, "Error, issuing timed out");
-                return;
-            }
-            resp.getWriter().println("OK");
+
+        Certificate c = Certificate.getBySerial(tserial);
+        if (c == null || c.getOwner() != u) {
+            resp.sendError(403, "Access Denied");
+            return;
+        }
+
+        Job job = c.revoke();
+        job.waitFor(60000);
+        if (c.getStatus() != CertificateStatus.REVOKED) {
+            resp.sendError(510, "Error, issuing timed out");
             return;
-        } catch (InterruptedException e) {
-            e.printStackTrace();
         }
+
+        resp.getWriter().println("OK");
+
     }
 }