From 09e2de14e84f8e59e946a40d193ccd959f311601 Mon Sep 17 00:00:00 2001
From: Thibault Duplessis <thibault.duplessis@gmail.com>
Date: Wed, 5 Dec 2018 14:44:41 +0700
Subject: [PATCH] log fewer fishnet client errors

---
 app/controllers/Fishnet.scala | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/app/controllers/Fishnet.scala b/app/controllers/Fishnet.scala
index ab85726820..99a9c1f049 100644
--- a/app/controllers/Fishnet.scala
+++ b/app/controllers/Fishnet.scala
@@ -49,10 +49,7 @@ object Fishnet extends LilaController {
   def keyExists(key: String) = Action.async { req =>
     api keyExists lila.fishnet.Client.Key(key) map {
       case true => Ok
-      case false =>
-        val ip = HTTPRequest.lastRemoteAddress(req)
-        logger.info(s"Unauthorized key: $key ip: $ip")
-        NotFound
+      case false => NotFound
     }
   }
 
@@ -68,11 +65,7 @@ object Fishnet extends LilaController {
           BadRequest(jsonError(JsError toJson err)).fuccess
         },
         data => api.authenticateClient(data, HTTPRequest lastRemoteAddress req) flatMap {
-          case Failure(msg) => {
-            val ip = HTTPRequest.lastRemoteAddress(req)
-            logger.info(s"Unauthorized key: ${data.fishnet.apikey} ip: $ip | ${msg.getMessage}")
-            Unauthorized(jsonError(msg.getMessage)).fuccess
-          }
+          case Failure(msg) => Unauthorized(jsonError(msg.getMessage)).fuccess
           case Success(client) => f(data)(client).map {
             case Right(Some(work)) => Accepted(Json toJson work)
             case Right(None) => NoContent