From f3509dfaeed1d7e82253c9bf082251ce35348c0e Mon Sep 17 00:00:00 2001 From: Anders Lindh Olsson Date: Fri, 8 May 2026 12:18:48 +0200 Subject: [PATCH] =?UTF-8?q?fix(web):=20drop=20not-found=20log=20noise=20?= =?UTF-8?q?=E2=80=94=20FINE=20without=20stack=20trace?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A misbehaving client hammering a non-existent channel generated thousands of WARNING entries, drowning out real server-side failures. 404s are client errors; the HTTP response is sufficient, and the audit log already records every lookup at INFO. FINE keeps the messages available under debug logging without polluting production logs. --- .../phoebus/channelfinder/web/v0/V0ExceptionHandler.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/org/phoebus/channelfinder/web/v0/V0ExceptionHandler.java b/src/main/java/org/phoebus/channelfinder/web/v0/V0ExceptionHandler.java index ef3e3a17..6dd7a7ec 100644 --- a/src/main/java/org/phoebus/channelfinder/web/v0/V0ExceptionHandler.java +++ b/src/main/java/org/phoebus/channelfinder/web/v0/V0ExceptionHandler.java @@ -33,21 +33,21 @@ public class V0ExceptionHandler { @ExceptionHandler(ChannelNotFoundException.class) @ResponseStatus(HttpStatus.NOT_FOUND) public ResponseStatusException handleChannelNotFound(ChannelNotFoundException ex) { - logger.log(Level.WARNING, ex.getMessage(), ex); + logger.log(Level.FINE, ex::getMessage); return new ResponseStatusException(HttpStatus.NOT_FOUND, ex.getMessage()); } @ExceptionHandler(TagNotFoundException.class) @ResponseStatus(HttpStatus.NOT_FOUND) public ResponseStatusException handleTagNotFound(TagNotFoundException ex) { - logger.log(Level.WARNING, ex.getMessage(), ex); + logger.log(Level.FINE, ex::getMessage); return new ResponseStatusException(HttpStatus.NOT_FOUND, ex.getMessage()); } @ExceptionHandler(PropertyNotFoundException.class) @ResponseStatus(HttpStatus.NOT_FOUND) public ResponseStatusException handlePropertyNotFound(PropertyNotFoundException ex) { - logger.log(Level.WARNING, ex.getMessage(), ex); + logger.log(Level.FINE, ex::getMessage); return new ResponseStatusException(HttpStatus.NOT_FOUND, ex.getMessage()); }