From 40fb669a9d2e95ef33f76d992d4fe0048083c883 Mon Sep 17 00:00:00 2001
From: Kelvin Chappell <kelvin.chappell@theguardian.com>
Date: Thu, 4 Jun 2015 09:51:47 +0100
Subject: [PATCH] Log Exception

---
 commercial/app/model/commercial/CapiAgent.scala | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/commercial/app/model/commercial/CapiAgent.scala b/commercial/app/model/commercial/CapiAgent.scala
index 769ca309c457..2b52f3afe231 100644
--- a/commercial/app/model/commercial/CapiAgent.scala
+++ b/commercial/app/model/commercial/CapiAgent.scala
@@ -1,12 +1,12 @@
 package model.commercial
 
-import common.AkkaAgent
+import common.{AkkaAgent, Logging}
 import model.Content
 
 import scala.concurrent.{ExecutionContext, Future}
 import scala.util.control.NonFatal
 
-object CapiAgent {
+object CapiAgent extends Logging {
 
   private lazy val shortUrlAgent = AkkaAgent[Map[String, Option[Content]]](Map.empty)
 
@@ -36,7 +36,9 @@ object CapiAgent {
       Lookup.contentByShortUrls(shortUrlIds) flatMap {
         addToCache
       } recoverWith {
-        case NonFatal(e) => Future.successful(cache)
+        case NonFatal(e) =>
+          log.error(s"Lookup failed: ${e.getMessage}")
+          Future.successful(cache)
       }
     }