Change the log levels

guardian · Apr 10, 2024 · dccb9c2 · dccb9c2
1 parent 70470e4
commit dccb9c2
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 11 deletions.
diff --git a/...e-for-later-user-deletion/src/main/scala/com/gu/sfl/userdeletion/UserDeletionLambda.scala b/...e-for-later-user-deletion/src/main/scala/com/gu/sfl/userdeletion/UserDeletionLambda.scala
@@ -21,7 +21,6 @@ object UserDeletionLambda extends Logging {
     messages.map{
       m =>
         val body = m.getBody
-        logger.debug(s"Message body: ${body}")
         val node = mapper.readTree(body)
         val user = mapper.readValue[UserDeleteMessage](node.get("Message").asText())
         sflDyanamo.deleteSavedArticleasForUser(user)

diff --git a/mobile-save-for-later/src/main/scala/com/gu/sfl/controller/FetchArticlesController.scala b/mobile-save-for-later/src/main/scala/com/gu/sfl/controller/FetchArticlesController.scala
@@ -17,7 +17,7 @@ class FetchArticlesController(fetchSavedArticles: FetchSavedArticles)(implicit e
      val futureResponse =  fetchSavedArticles.retrieveForUser(lambdaRequest.headers).map {
        case Right(syncedPrefs) =>
          syncedPrefs.savedArticles.foreach ( sp =>
-            logger.debug(s"Returning found: ${sp.articles.size} articles")
+            logger.info(s"Returning found: ${sp.articles.size} articles")
          )
          okSyncedPrefsResponse(syncedPrefs)
        case Left(error) =>

diff --git a/mobile-save-for-later/src/main/scala/com/gu/sfl/savedarticles/FetchSavedArticles.scala b/mobile-save-for-later/src/main/scala/com/gu/sfl/savedarticles/FetchSavedArticles.scala
@@ -33,20 +33,20 @@ class FetchSavedArticlesImpl(identityService: IdentityService, savedArticlesPers
     } yield {
       identityService.userFromRequest(identityHeaders, List(readSelf)).transformWith{
         case Success(Some(userId)) =>
-          logger.debug(s"Got user id ${userId} from identity")
+          logger.info(s"Got user id ${userId} from identity")
           Future.successful(wrapSavedArticles(userId, savedArticlesPersistence.read(userId)))
         case Success(_) =>
-          logger.debug(s"No user found for AccessToken ${identityHeaders.accessToken}")
+          logger.error(s"No user found for AccessToken ${identityHeaders.accessToken}")
           Future.successful(Left(new UserNotFoundError("Could not retrieve a user id")))
         case Failure(OktaValidationException(e)) =>
-          logger.debug(s"Error retrieving userId from okta oauth token")
+          logger.error(s"Error retrieving userId from okta oauth token")
           Future.successful(Left(OktaOauthValidationError(e.message, e)))
         case Failure(_) =>
-          logger.debug(s"Error retrieving userId for: token: ${identityHeaders.accessToken}")
+          logger.error(s"Error retrieving userId for: token: ${identityHeaders.accessToken}")
           Future.successful(Left(new IdentityServiceError("Could not get a response from the id api")))
       }
     }).getOrElse{
-      logger.debug(s"Could not retrieve identity headers")
+      logger.error(s"Could not retrieve identity headers")
       Future.successful(Left(MissingAccessTokenError("No access token on request")))
     }
   }

diff --git a/mobile-save-for-later/src/main/scala/com/gu/sfl/savedarticles/UpdateSavedArticles.scala b/mobile-save-for-later/src/main/scala/com/gu/sfl/savedarticles/UpdateSavedArticles.scala
@@ -24,16 +24,16 @@ class UpdateSavedArticlesImpl(identityService: IdentityService, savedArticlesMer
       val eventualMaybeString = identityService.userFromRequest(identityHeaders, List(updateSelf))
       eventualMaybeString transformWith {
           case Success(Some(userId)) =>
-            logger.debug(s"Storing ${savedArticles.numberOfArticles} articles for user $userId")
+            logger.info(s"Storing ${savedArticles.numberOfArticles} articles for user $userId")
             Future.successful(savedArticlesMerger.updateWithRetryAndMerge(userId, savedArticles))
           case Success(_) =>
-            logger.debug(s"Could not retrieve a user id for token: ${identityHeaders.auth}")
+            logger.error(s"Could not retrieve a user id for token: ${identityHeaders.auth}")
             Future.successful(Left(new UserNotFoundError("Could not retrieve a user id")))
           case Failure(OktaValidationException(e)) =>
-            logger.debug(s"Error retrieving userId from okta oauth token")
+            logger.error(s"Error retrieving userId from okta oauth token")
             Future.successful(Left(OktaOauthValidationError(e.message, e)))
           case Failure(_) =>
-            logger.debug(s"Error retrieving userId for: token: ${identityHeaders.accessToken}")
+            logger.error(s"Error retrieving userId for: token: ${identityHeaders.accessToken}")
             Future.successful(Left(new IdentityServiceError("Could not retrieve a user from the id api")))
         }
      }).getOrElse(Future.successful(Left(new MissingAccessTokenError("No access token on request"))))