From 1a6d98fb54da85f1d8a79102a1ec0bd5aa42205d Mon Sep 17 00:00:00 2001 From: Discookie Date: Thu, 11 Jun 2026 07:42:02 +0000 Subject: [PATCH 1/2] Change all pre-endpoint product/version failures to ProductNotFoundError --- web/server/codechecker_server/server.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/web/server/codechecker_server/server.py b/web/server/codechecker_server/server.py index b06ef3a005..600c116f51 100644 --- a/web/server/codechecker_server/server.py +++ b/web/server/codechecker_server/server.py @@ -479,8 +479,8 @@ def do_POST(self): error_msg = \ "Requested CodeCheckerService on a " \ f"nonexistent product: '{product_endpoint}'." - LOG.error(error_msg) - raise ValueError(error_msg) + LOG.debug(error_msg) + raise ProductNotFoundError(error_msg) if product_endpoint: # The current request came through a @@ -501,9 +501,9 @@ def do_POST(self): else: LOG.debug("This API endpoint does not exist.") error_msg = f"No API endpoint named '{self.path}'." - raise ValueError(error_msg) + raise ProductNotFoundError(error_msg) else: - raise ValueError( + raise ProductNotFoundError( f"API version {major_version} not supported") else: From d8dc51dbfde9f13b15fc69ce8c011f54f468c1d2 Mon Sep 17 00:00:00 2001 From: Discookie Date: Tue, 30 Jun 2026 11:59:55 +0000 Subject: [PATCH 2/2] Remove redundant logs --- web/server/codechecker_server/server.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/web/server/codechecker_server/server.py b/web/server/codechecker_server/server.py index 600c116f51..53ce4765e2 100644 --- a/web/server/codechecker_server/server.py +++ b/web/server/codechecker_server/server.py @@ -476,11 +476,9 @@ def do_POST(self): elif request_endpoint == "CodeCheckerService": # This endpoint is a product's report_server. if not product: - error_msg = \ - "Requested CodeCheckerService on a " \ - f"nonexistent product: '{product_endpoint}'." - LOG.debug(error_msg) - raise ProductNotFoundError(error_msg) + raise ProductNotFoundError( + "Requested CodeCheckerService on a " + f"nonexistent product: '{product_endpoint}'.") if product_endpoint: # The current request came through a @@ -499,9 +497,8 @@ def do_POST(self): self.server.context) processor = ReportAPI_v6.Processor(acc_handler) else: - LOG.debug("This API endpoint does not exist.") - error_msg = f"No API endpoint named '{self.path}'." - raise ProductNotFoundError(error_msg) + raise ProductNotFoundError( + f"No API endpoint named '{self.path}'.") else: raise ProductNotFoundError( f"API version {major_version} not supported")