Mercurial > dive4elements > framework
changeset 40:af22d4de275c
Log RuntimeExceptions in REST calls to log4j.
artifacts/trunk@112 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author | Sascha L. Teichmann <sascha.teichmann@intevation.de> |
---|---|
date | Wed, 23 Sep 2009 08:27:35 +0000 (2009-09-23) |
parents | 075cb3778d71 |
children | 5e4bc24ea438 |
files | Changelog artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactOutResource.java artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactResource.java artifact-database/src/main/java/de/intevation/artifactdatabase/rest/BaseResource.java artifact-database/src/main/java/de/intevation/artifactdatabase/rest/CreateResource.java artifact-database/src/main/java/de/intevation/artifactdatabase/rest/FactoriesResource.java |
diffstat | 6 files changed, 103 insertions(+), 42 deletions(-) [+] |
line wrap: on
line diff
--- a/Changelog Mon Sep 21 11:56:10 2009 +0000 +++ b/Changelog Wed Sep 23 08:27:35 2009 +0000 @@ -1,5 +1,18 @@ -2009-09-21 Tim Englich <tim.englich@intevation.de> - * Added some IDE-specific files to the Ignorelist. +2009-09-23 Sascha L. Teichmann <sascha.teichmann@intevation.de> + + * artifact-database/src/main/java/de/intevation/artifactdatabase/rest/BaseResource.java: + New. Base class to log RuntimeExceptions to log4j error log. + + * artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactResource.java, + artifact-database/src/main/java/de/intevation/artifactdatabase/rest/FactoriesResource.java, + artifact-database/src/main/java/de/intevation/artifactdatabase/rest/CreateResource.java, + artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactOutResource.java: + Adjusted to extend from BaseResource. + +2009-09-21 Tim Englich <tim.englich@intevation.de> + + * Added some IDE-specific files to the Ignorelist. + 2009-09-18 Sascha L. Teichmann <sascha.teichmann@intevation.de> * artifact-database/doc/schema.sql: removed.
--- a/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactOutResource.java Mon Sep 21 11:56:10 2009 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactOutResource.java Wed Sep 23 08:27:35 2009 +0000 @@ -2,9 +2,7 @@ import org.apache.log4j.Logger; -import org.restlet.resource.Get; -import org.restlet.resource.Post; -import org.restlet.resource.ServerResource; +import org.restlet.resource.ResourceException; import org.restlet.data.Request; import org.restlet.data.Response; @@ -30,7 +28,7 @@ * @author Sascha L. Teichmann (sascha.teichmann@intevation) */ public class ArtifactOutResource -extends ServerResource +extends BaseResource { public static final String PATH = "/artifact/{uuid}/{type}"; @@ -41,13 +39,9 @@ private static Logger logger = Logger.getLogger(ArtifactOutResource.class); - @Post - public Representation represent() { - - Request request = getRequest(); - - Representation requestRepr = request.getEntity(); - + protected Representation innerPost(Representation requestRepr) + throws ResourceException + { Document inputDocument = null; try { DomRepresentation input = new DomRepresentation(requestRepr); @@ -63,6 +57,8 @@ ArtifactDatabase db = (ArtifactDatabase)getContext() .getAttributes().get("database"); + Request request = getRequest(); + String identifier = (String)request.getAttributes().get("uuid"); if (logger.isDebugEnabled()) {
--- a/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactResource.java Mon Sep 21 11:56:10 2009 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/ArtifactResource.java Wed Sep 23 08:27:35 2009 +0000 @@ -1,8 +1,6 @@ package de.intevation.artifactdatabase.rest; -import org.restlet.resource.Get; -import org.restlet.resource.Post; -import org.restlet.resource.ServerResource; +import org.restlet.resource.ResourceException; import org.restlet.representation.Representation; import org.restlet.representation.EmptyRepresentation; @@ -31,7 +29,7 @@ * @author Sascha L. Teichmann (sascha.teichmann@intevation) */ public class ArtifactResource -extends ServerResource +extends BaseResource { private static Logger logger = Logger.getLogger(ArtifactResource.class); @@ -47,9 +45,9 @@ public static final String ADVANCE = "advance"; public static final String FEED = "feed"; - @Get - public Representation represent() { - + protected Representation innerGet() + throws ResourceException + { Request request = getRequest(); String identifier = (String)request.getAttributes().get("uuid"); @@ -105,11 +103,7 @@ return new DomRepresentation(MediaType.APPLICATION_XML, out); } - @Post - public Representation representPost() { - Request request = getRequest(); - - Representation requestRepr = request.getEntity(); + protected Representation innerPost(Representation requestRepr) { Document inputDocument = null; try { @@ -135,6 +129,8 @@ return new EmptyRepresentation(); } + Request request = getRequest(); + String identifier = (String)request.getAttributes().get("uuid"); ArtifactDatabase db = (ArtifactDatabase)getContext()
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/BaseResource.java Wed Sep 23 08:27:35 2009 +0000 @@ -0,0 +1,63 @@ +package de.intevation.artifactdatabase.rest; + +import org.apache.log4j.Logger; + +import org.restlet.resource.ServerResource; +import org.restlet.resource.ResourceException; + +import org.restlet.representation.Representation; + +/** + * @author Sascha L. Teichmann (sascha.teichmann@intevation) + */ +public class BaseResource +extends ServerResource +{ + private static Logger logger = Logger.getLogger(BaseResource.class); + + public BaseResource() { + } + + protected Representation post(Representation requestRepr) + throws ResourceException + { + try { + return innerPost(requestRepr); + } + catch (ResourceException re) { + throw re; + } + catch (RuntimeException re) { + logger.error(re.getLocalizedMessage(), re); + throw re; + } + } + + protected Representation innerPost(Representation requestRepr) + throws ResourceException + { + return super.post(requestRepr); + } + + protected Representation get() + throws ResourceException + { + try { + return innerGet(); + } + catch (ResourceException re) { + throw re; + } + catch (RuntimeException re) { + logger.error(re.getLocalizedMessage(), re); + throw re; + } + } + + protected Representation innerGet() + throws ResourceException + { + return super.get(); + } +} +// vim:set ts=4 sw=4 si et sta sts=4 fenc=utf8:
--- a/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/CreateResource.java Mon Sep 21 11:56:10 2009 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/CreateResource.java Wed Sep 23 08:27:35 2009 +0000 @@ -4,10 +4,8 @@ import org.restlet.ext.xml.DomRepresentation; -import org.restlet.resource.Post; -import org.restlet.resource.ServerResource; +import org.restlet.resource.ResourceException; -import org.restlet.data.Request; import org.restlet.data.Response; import org.restlet.data.Status; @@ -30,7 +28,7 @@ * @author Sascha L. Teichmann (sascha.teichmann@intevation) */ public class CreateResource -extends ServerResource +extends BaseResource { private static Logger logger = Logger.getLogger(CreateResource.class); @@ -42,13 +40,9 @@ public static final String NO_ARTIFACT_CREATED = "No artifact created"; - @Post - public Representation represent() { - - Request request = getRequest(); - - Representation requestRepr = request.getEntity(); - + protected Representation innerPost(Representation requestRepr) + throws ResourceException + { Document inputDocument = null; try { DomRepresentation input = new DomRepresentation(requestRepr);
--- a/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/FactoriesResource.java Mon Sep 21 11:56:10 2009 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/rest/FactoriesResource.java Wed Sep 23 08:27:35 2009 +0000 @@ -5,8 +5,7 @@ import org.restlet.data.MediaType; -import org.restlet.resource.Get; -import org.restlet.resource.ServerResource; +import org.restlet.resource.ResourceException; import org.restlet.ext.xml.DomRepresentation; @@ -24,15 +23,15 @@ * @author Sascha L. Teichmann (sascha.teichmann@intevation) */ public class FactoriesResource -extends ServerResource +extends BaseResource { private static Logger logger = Logger.getLogger(FactoriesResource.class); public static final String PATH = "/factories"; - @Get - public Representation represent() { - + protected Representation innerGet() + throws ResourceException + { Document document = XMLUtils.newDocument(); ElementCreator ec = new ElementCreator(