[jboss-svn-commits] JBL Code SVN: r33403 - labs/jbossrules/trunk/drools-server/drools-server-app/src/main/java/org/drools/server/service.

[jboss-svn-commits at lists.jboss.org]

Author: baunax
Date: 2010-06-08 10:48:23 -0400 (Tue, 08 Jun 2010)
New Revision: 33403

Modified:
   labs/jbossrules/trunk/drools-server/drools-server-app/src/main/java/org/drools/server/service/KnowledgeServiceRest.java
Log:
JBRULES-2540: drools-server: change exceptions & update config
	- changed to log the full exceptions.

Modified: labs/jbossrules/trunk/drools-server/drools-server-app/src/main/java/org/drools/server/service/KnowledgeServiceRest.java
===================================================================
--- labs/jbossrules/trunk/drools-server/drools-server-app/src/main/java/org/drools/server/service/KnowledgeServiceRest.java	2010-06-08 14:42:26 UTC (rev 33402)
+++ labs/jbossrules/trunk/drools-server/drools-server-app/src/main/java/org/drools/server/service/KnowledgeServiceRest.java	2010-06-08 14:48:23 UTC (rev 33403)
@@ -26,22 +26,20 @@
     @Path("/execute")
     public Response execute(@FormParam("command") String command) {
         if ( command == null || command.length() == 0 ) {
-            logger.error( "Invalid or null command " + command );
+            logger.error( "Invalid or null command: {}", command );
             return Response.status( Status.BAD_REQUEST ).build();
         }
-        String response;
         try {
-            response = getService().executeCommand( command );
+            ResponseBuilder builder = 
+            	Response.ok( getService().executeCommand( command ), "application/xml" );
+            return builder.build();
         } catch ( RuntimeException e ) {
-            logger.error( e.getMessage() );
+            logger.error( e.getMessage(), e );
             return Response.status( Status.BAD_REQUEST ).build();
         } catch ( Exception e ) {
-            logger.error( e.getMessage() );
+            logger.error( e.getMessage(), e );
             return Response.status( Status.CONFLICT ).build();
         }
-        ResponseBuilder builder = Response.ok( response,
-                                               "application/xml" );
-        return builder.build();
     }
 
     public void setService(KnowledgeService service) {
@@ -51,5 +49,4 @@
     public KnowledgeService getService() {
         return service;
     }
-
 }