chore: fast return for invalid request of node health (#5762)

Co-authored-by: command-bot <>

prdoc/pr_5762.prdoc

@@ -0,0 +1,10 @@
+title: Fast return for invalid request of node health
+
+doc:
+  - audience: Node Dev
+    description: |
+      Return directly when invalid request for node health api
+
+crates:
+  - name: sc-rpc-server
+    bump: patch

substrate/client/rpc-servers/src/middleware/node_health.rs

@@ -98,17 +98,17 @@ where
 		let fut = self.0.call(req);
 
 		async move {
-			let res = fut.await.map_err(|err| err.into())?;
-
 			Ok(match maybe_intercept {
 				InterceptRequest::Deny =>
 					http_response(StatusCode::METHOD_NOT_ALLOWED, HttpBody::empty()),
-				InterceptRequest::No => res,
+				InterceptRequest::No => fut.await.map_err(|err| err.into())?,
 				InterceptRequest::Health => {
+					let res = fut.await.map_err(|err| err.into())?;
 					let health = parse_rpc_response(res.into_body()).await?;
 					http_ok_response(serde_json::to_string(&health)?)
 				},
 				InterceptRequest::Readiness => {
+					let res = fut.await.map_err(|err| err.into())?;
 					let health = parse_rpc_response(res.into_body()).await?;
 					if (!health.is_syncing && health.peers > 0) || !health.should_have_peers {
 						http_ok_response(HttpBody::empty())