Merge pull request #21 from osmosis-labs/adam/downstream-websocket-lo…

…g-noise fix(rpc): Make websocket's log on success be in Debug, not info (cometbft#2788)
osmosis-labs · Apr 15, 2024 · 2faae8c · 2faae8c
1 parent dc6313c
commit 2faae8c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/unreleased/improvements/2788-move-ws-info-log-to-debug b/.changelog/unreleased/improvements/2788-move-ws-info-log-to-debug
@@ -0,0 +1 @@
+Move the websockets info log for successful replies to debug.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+* [#21](https://github.com/osmosis-labs/cometbft/pull/21) Move websocket logs to Debug
+
 ## v0.37.4-v24-osmo-2
 
 * [#20](https://github.com/osmosis-labs/cometbft/pull/20) Fix the rollback command

diff --git a/rpc/jsonrpc/server/ws_handler.go b/rpc/jsonrpc/server/ws_handler.go
@@ -381,7 +381,7 @@ func (wsc *wsConnection) readRoutine() {
 			returns := rpcFunc.f.Call(args)
 
 			// TODO: Need to encode args/returns to string if we want to log them
-			wsc.Logger.Info("WSJSONRPC", "method", request.Method)
+			wsc.Logger.Debug("WSJSONRPC", "method", request.Method)
 
 			result, err := unreflectResult(returns)
 			if err != nil {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Move the websockets info log for successful replies to debug.