Merge pull request #6340 from onflow/petera/reduce-access-logging

[Access] Reduce logging on some access endpoints
onflow · Aug 15, 2024 · a21eca2 · a21eca2
2 parents 39ae577 + 0a38a3c
commit a21eca2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/engine/access/rest/middleware/logging.go b/engine/access/rest/middleware/logging.go
@@ -23,9 +23,6 @@ func LoggingMiddleware(logger zerolog.Logger) mux.MiddlewareFunc {
 			// continue to the next handler
 			inner.ServeHTTP(respWriter, req)
 			log := logger.Info()
-			if respWriter.statusCode != http.StatusOK {
-				log = logger.Error()
-			}
 			log.Str("method", req.Method).
 				Str("uri", req.RequestURI).
 				Str("client_ip", req.RemoteAddr).

diff --git a/engine/access/rpc/backend/backend_scripts.go b/engine/access/rpc/backend/backend_scripts.go
@@ -198,7 +198,7 @@ func (b *backendScripts) executeScriptLocally(
 			logEvent.Msg("script failed to execute locally")
 
 		default:
-			lg.Error().Err(err).Msg("script execution failed")
+			lg.Debug().Err(err).Msg("script execution failed")
 			b.metrics.ScriptExecutionErrorLocal()
 		}