From 116f762cb8a909751fd54491b4afa994f76bcb2e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Istv=C3=A1n=20So=C3=B3s?= <isoos@users.noreply.github.com>
Date: Thu, 23 May 2024 15:50:08 +0200
Subject: [PATCH] Log only >=500 status codes when search fails. (#7749)

---
 app/lib/frontend/handlers/listing.dart | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/app/lib/frontend/handlers/listing.dart b/app/lib/frontend/handlers/listing.dart
index 138b137ba8..eda837b2ba 100644
--- a/app/lib/frontend/handlers/listing.dart
+++ b/app/lib/frontend/handlers/listing.dart
@@ -80,7 +80,10 @@ Future<shelf.Response> _packagesHandlerHtmlCore(shelf.Request request) async {
     rateLimitKey: request.sourceIp,
   );
   final int totalCount = searchResult.totalCount;
-  if (searchResult.errorMessage != null) {
+  final errorMessage = searchResult.errorMessage;
+  final statusCode =
+      searchResult.statusCode ?? (errorMessage == null ? 200 : 500);
+  if (errorMessage != null && statusCode >= 500) {
     _logger.severe('[pub-search-not-working] ${searchResult.errorMessage}');
   }
 
@@ -93,9 +96,7 @@ Future<shelf.Response> _packagesHandlerHtmlCore(shelf.Request request) async {
       messageFromBackend: searchResult.errorMessage,
       openSections: openSections,
     ),
-    status: searchResult.errorMessage == null
-        ? 200
-        : (searchResult.statusCode ?? 500),
+    status: statusCode,
   );
   _searchOverallLatencyTracker.add(sw.elapsed);
   return result;