fix(volo-http): correct server handle timings

Joshuahoky · Joshuahoky · commit af2f51d40e03 · 2026-04-08T14:25:38.000+08:00
diff --git a/volo-http/src/server/handler.rs b/volo-http/src/server/handler.rs
@@ -74,7 +74,10 @@ macro_rules! impl_handler {
                     Ok(value) => value,
                     Err(rejection) => return rejection.into_response(),
                 };
-                self($($ty,)* $last).await.into_response()
+                cx.stats.record_handle_start();
+                let result = self($($ty,)* $last).await;
+                cx.stats.record_handle_finish();
+                result.into_response()
             }
         }
     };
diff --git a/volo-http/src/server/mod.rs b/volo-http/src/server/mod.rs
@@ -534,14 +534,12 @@ where
                 let mut cx = ServerContext::new(service.peer);
                 cx.rpc_info_mut().set_config(service.config);
                 let span = service.span_provider.on_serve(&cx);
-                cx.stats.record_handle_start();
                 let resp: http::Response<Body> = service
                     .inner
                     .call(&mut cx, req.map(Body::from_incoming))
                     .instrument(span)
                     .await
                     .into_response();
-                cx.stats.record_handle_finish();
                 service.span_provider.leave_serve(&cx);
                 Ok(resp)
             }),

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,10 @@ macro_rules! impl_handler {`
`74`	`74`	`Ok(value) => value,`
`75`	`75`	`Err(rejection) => return rejection.into_response(),`
`76`	`76`	`};`
`77`		`- self($($ty,)* $last).await.into_response()`
	`77`	`+ cx.stats.record_handle_start();`
	`78`	`+ let result = self($($ty,)* $last).await;`
	`79`	`+ cx.stats.record_handle_finish();`
	`80`	`+ result.into_response()`
`78`	`81`	`}`
`79`	`82`	`}`
`80`	`83`	`};`