let lichobile pass the UA test

lichess-org · Jan 18, 2025 · 434ef97 · 434ef97
1 parent c4b592f
commit 434ef97
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/modules/security/src/main/UserAgentParser.scala b/modules/security/src/main/UserAgentParser.scala
@@ -39,10 +39,12 @@ object UserAgentParser:
     def isSuspicious(req: RequestHeader): Boolean = HTTPRequest.userAgent(req).forall(isSuspicious)
 
     def isSuspicious(ua: UA): Boolean =
-      val str = ua.value.take(200).toLowerCase
-      str.lengthIs < 30 || isMacOsEdge(str) || !looksNormal(str)
+      !isLichobile(ua) && {
+        val str = ua.value.take(200).toLowerCase
+        str.lengthIs < 30 || isMacOsEdge(str) || !popularBrowser(str)
+      }
 
-    private def looksNormal(ua: String) =
+    private def popularBrowser(ua: String) =
       val sections = ua.split(' ')
       sections.exists: s =>
         isRecentChrome(s) || isLastWindows8Chrome(s) || isRecentFirefox(s) || isRecentSafari(s)
@@ -62,3 +64,5 @@ object UserAgentParser:
     private def isLastWindows8Chrome(s: String) = s.startsWith("chrome/109.")
 
     private def isMacOsEdge(ua: String) = ua.contains("macintosh") && ua.contains("edg/")
+
+    private def isLichobile(ua: UA) = ua.value.contains("Lichobile/")