From a5db67d0cb4e0c4835a18860f7cbee1519ca2ca7 Mon Sep 17 00:00:00 2001
From: Colin Alworth <colin@colinalworth.com>
Date: Mon, 28 Oct 2024 11:39:32 -0500
Subject: [PATCH] Deframe failures should be logged on the server as warnings

This brings grpc-servlet in line with the grpc-netty implementation found
in NettyServerStream.TransportState.
---
 .../src/main/java/io/grpc/servlet/ServletServerStream.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/servlet/src/main/java/io/grpc/servlet/ServletServerStream.java b/servlet/src/main/java/io/grpc/servlet/ServletServerStream.java
index b7ad6e0dec..5d11abf0f9 100644
--- a/servlet/src/main/java/io/grpc/servlet/ServletServerStream.java
+++ b/servlet/src/main/java/io/grpc/servlet/ServletServerStream.java
@@ -154,8 +154,8 @@ final class ServletServerStream extends AbstractServerStream {
 
     @Override
     public void deframeFailed(Throwable cause) {
-      if (logger.isLoggable(FINE)) {
-        logger.log(FINE, String.format("[{%s}] Exception processing message", logId), cause);
+      if (logger.isLoggable(WARNING)) {
+        logger.log(WARNING, String.format("[{%s}] Exception processing message", logId), cause);
       }
       cancel(Status.fromThrowable(cause));
     }