Handle parsing error instead of logging errors (#1768)

lavanet · Nov 18, 2024 · 0625d61 · 0625d61
1 parent 8803889
commit 0625d61
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/protocol/chainlib/jsonRPC.go b/protocol/chainlib/jsonRPC.go
@@ -432,6 +432,10 @@ func (apil *JsonRPCChainListener) Serve(ctx context.Context, cmdFlags common.Con
 				return fiberCtx.Status(fiber.StatusOK).JSON(common.JsonRpcMethodNotFoundError)
 			}
 
+			if _, ok := err.(*json.SyntaxError); ok {
+				return fiberCtx.Status(fiber.StatusBadRequest).JSON(common.JsonRpcParseError)
+			}
+
 			// Get unique GUID response
 			errMasking := apil.logger.GetUniqueGuidResponseForError(err, msgSeed)
 

diff --git a/protocol/common/return_errors.go b/protocol/common/return_errors.go
@@ -36,6 +36,16 @@ var JsonRpcRateLimitError = JsonRPCErrorMessage{
 	},
 }
 
+var JsonRpcParseError = JsonRPCErrorMessage{
+	JsonRPC: "2.0",
+	Id:      -1,
+	Error: JsonRPCError{
+		Code:    -32700,
+		Message: "Parse error",
+		Data:    "Failed to parse the request body as JSON",
+	},
+}
+
 var JsonRpcSubscriptionNotFoundError = JsonRPCErrorMessage{
 	JsonRPC: "2.0",
 	Id:      1,