no logResponse needed (#51)

DANS-KNAW · Apr 15, 2019 · 0b3d009 · 0b3d009
1 parent bb24019
commit 0b3d009
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -75,7 +75,7 @@
         <dependency>
             <groupId>nl.knaw.dans.lib</groupId>
             <artifactId>dans-scala-lib_2.12</artifactId>
-            <version>1.5.0</version>
+            <version>1.6.0</version>
         </dependency>
 
         <!-- EASY -->

diff --git a/src/main/scala/nl/knaw/dans/easy/agreement/AgreementCreatorServlet.scala b/src/main/scala/nl/knaw/dans/easy/agreement/AgreementCreatorServlet.scala
@@ -29,10 +29,11 @@ import scala.util.{ Failure, Success, Try }
 class AgreementCreatorServlet(app: AgreementCreatorApp) extends ScalatraServlet
   with ServletLogger
   with PlainLogFormatter
+  with LogResponseBodyOnError
   with DebugEnhancedLogging {
 
   get("/") {
-    Ok(s"Agreement Creator Service running v${ app.version }.").logResponse
+    Ok(s"Agreement Creator Service running v${ app.version }.")
   }
 
   post("/create") {
@@ -51,7 +52,7 @@ class AgreementCreatorServlet(app: AgreementCreatorApp) extends ScalatraServlet
       case nse: NoSuchElementException => NotFound(nse.getMessage)
       case iae: IllegalArgumentException => BadRequest(iae.getMessage)
       case t => InternalServerError(t.getMessage)
-    }.logResponse
+    }
   }
 
   private def validateDatasetIdExistsInFedora(pars: Params): Try[Unit] = {