API-1888 - Add trace requests and debug log when DES does not have data

hmrc · Sep 12, 2016 · 351628d · 351628d
1 parent 06c8d80
commit 351628d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/app/uk/gov/hmrc/openidconnect/userinfo/connectors/DesConnector.scala b/app/uk/gov/hmrc/openidconnect/userinfo/connectors/DesConnector.scala
@@ -16,6 +16,7 @@
 
 package uk.gov.hmrc.openidconnect.userinfo.connectors
 
+import play.api.Logger
 import uk.gov.hmrc.openidconnect.userinfo.config.{WSHttp, AppContext}
 import uk.gov.hmrc.openidconnect.userinfo.domain.DesUserInfo
 import uk.gov.hmrc.play.config.ServicesConfig
@@ -40,7 +41,9 @@ trait DesConnector {
       "Environment" -> desEnvironment)
 
     http.GET[DesUserInfo](s"$serviceUrl/pay-as-you-earn/individuals/${withoutSuffix(nino)}")(implicitly[HttpReads[DesUserInfo]], newHc) map (Some(_)) recover {
-      case _: NotFoundException | _: BadRequestException => None
+      case _: NotFoundException | _: BadRequestException =>
+        Logger.debug(s"User information for nino $nino is not available in DES")
+        None
     }
   }
 

diff --git a/app/uk/gov/hmrc/openidconnect/userinfo/services/UserInfoService.scala b/app/uk/gov/hmrc/openidconnect/userinfo/services/UserInfoService.scala
@@ -16,6 +16,7 @@
 
 package uk.gov.hmrc.openidconnect.userinfo.services
 
+import play.api.Logger
 import uk.gov.hmrc.openidconnect.userinfo.connectors.{AuthConnector, DesConnector}
 import uk.gov.hmrc.openidconnect.userinfo.data.UserInfoGenerator
 import uk.gov.hmrc.openidconnect.userinfo.domain._
@@ -42,7 +43,9 @@ trait LiveUserInfoService extends UserInfoService {
     } yield userInfo
 
     future map (Some(_)) recover {
-      case NinoNotFoundException() => None
+      case NinoNotFoundException() =>
+        Logger.debug("Nino not present in Bearer Token")
+        None
     }
   }
 }

diff --git a/conf/application.conf b/conf/application.conf
@@ -251,7 +251,7 @@ Prod {
 
     auditing {
         enabled=true
-        traceRequests=false
+        traceRequests=true
         consumer {
             baseUri {
                 host = datastream.service