From f3bd0aa5924e793090ed06c6d4a5800bf87c1384 Mon Sep 17 00:00:00 2001 From: Aron Roberts Date: Thu, 30 Apr 2015 21:03:38 -0700 Subject: [PATCH] CSPACE-6525: Log, and return in response, certain server-side errors when importing. --- .../services/imports/ImportsResource.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/services/imports/service/src/main/java/org/collectionspace/services/imports/ImportsResource.java b/services/imports/service/src/main/java/org/collectionspace/services/imports/ImportsResource.java index 94010f7cf..296e4eb86 100644 --- a/services/imports/service/src/main/java/org/collectionspace/services/imports/ImportsResource.java +++ b/services/imports/service/src/main/java/org/collectionspace/services/imports/ImportsResource.java @@ -203,22 +203,22 @@ public class ImportsResource extends AbstractCollectionSpaceResourceImpl ctx = createServiceContext(ui); + ServiceContext ctx = createServiceContext(ui); int timeout = ctx.getTimeoutSecs(); // gets it from query param 'impTimout' or uses default if no query param specified // InputSource inputSource = payloadToInputSource(xmlPayload); // result = createFromInputSource(inputSource); String inputFilename = payloadToFilename(xmlPayload); result = createFromFilename(inputFilename, timeout); - rb = Response.ok(); } catch (Exception e) { - result = Tools.errorToString(e, true); - rb = Response - .status(Response.Status.INTERNAL_SERVER_ERROR); - } - rb.entity(result); - return rb.build(); + result = e.getMessage(); + logger.error(result); + rb = Response.status(Response.Status.INTERNAL_SERVER_ERROR); + } finally { + rb.entity(result); + return rb.build(); + } } public static String createFromInputSource(InputSource inputSource, -- 2.47.3